-
Notifications
You must be signed in to change notification settings - Fork 858
[Universal] RenderPass merging #4099
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
ddbee5f
enabled MSAA when the RenderPass API is used. Resolving implicitly to…
manuele-bonanno 8f7db74
Initial lazy merging prototype
manuele-bonanno fe5cadb
Lazy merging prototype: all the needed attachments are now added to t…
manuele-bonanno e50c046
initial work to add lazy merging to the deferred renderer (doesn't ha…
manuele-bonanno 3ba7c7b
MRT attachments merging improvements
manuele-bonanno b66f183
fix for a crash because of incomplete Targets setup for merged render…
manuele-bonanno 422ed1b
change m_ShadowmapWidth/Height variables to use renderTargetWidth
jonuuukas 9bdd3ef
Merge branch 'universal/render-pass-conversion' of ssh://github.com/U…
jonuuukas 7d3c5ba
m_ActiveRenderPassQueue[i] to var renderPass
jonuuukas 6f5d3f0
changing hash ctor logic to merge width/height and include depth rtID
jonuuukas 7dc34fa
fix setlastpass flag loop
jonuuukas 72d51d2
NativeArray memory leak fix
manuele-bonanno 73d666d
refactoring to remove duplicated code
manuele-bonanno d86b828
fix non-renderpass passes getting hashes and etc.
jonuuukas ac46e52
more cleaning up, moving RenderPass code in separate functions for cl…
manuele-bonanno bc45166
lists to arrays and other gc.alloc related stuff
jonuuukas 7b828d8
Merge branch 'universal/render-pass-conversion' of ssh://github.com/U…
jonuuukas 44d350e
Merge branch 'master' into universal/render-pass-conversion
jonuuukas e57a3ed
merge leftovers
jonuuukas 432dc6b
moving all the RenderPass API specific code in separate methods as a …
manuele-bonanno bf030cb
remove Configure in SetLastPassFlag
jonuuukas f7d061a
refatoring (in progress): moving all the RenderPass API specific code…
manuele-bonanno 1b0b6bf
fixing depth-only offscreen camera and exposing usesNativeRenderPass
jonuuukas 6a6dd1f
Merge branch 'universal/render-pass-conversion' of ssh://github.com/U…
jonuuukas f8a8e62
merge leftovers
jonuuukas d84f779
more cleaning up and refactoring
manuele-bonanno ef3e215
splitting initial scene index setup from RenderPass data, and moved R…
manuele-bonanno eb3514d
Revert "splitting initial scene index setup from RenderPass data, and…
manuele-bonanno b89f1e8
fix for RenderPass MRT attachments being setup when the RenderPass AP…
manuele-bonanno ebd2d04
adding profiler samples to NativeRenderPass code
jonuuukas 82d7d6b
moving more RenderPass related code in NativeRenderPass.cs
manuele-bonanno 304a9ab
removed any direct references to RenderPass frame data outside of Nat…
manuele-bonanno 391784c
fixing targetTexture != null and depthOnly pass cornercase with wrong…
jonuuukas 617322d
NativeRenderPass from static to partial class conversion
jonuuukas 8a00998
add RenderPassDescriptor to contain dimensions and etc.
jonuuukas a62901e
add GetDefaultGraphicsFormat for RenderPass attachment descriptors
jonuuukas bc7a4b8
remove isFirstMergeable and isLastMergeable and add comments
jonuuukas 2ee17b8
moving some stuff and removing some parameters for methods due to par…
jonuuukas 073e77e
some more cleanup
jonuuukas 279d157
more cleanup: making methods non static and reducing the input parame…
manuele-bonanno c5d86db
renamed attachmentIndices in m_InputAttachmentIndices
manuele-bonanno f1c6990
more cleanup: removing unnecessary statics
manuele-bonanno e2c0ebe
improved subpass merging
manuele-bonanno bcc0638
more renaming of methods
manuele-bonanno 276e7d8
renaming more stuff for consistency with previous renamings
manuele-bonanno eb5be47
renderPass MRT clear logic fixed
jonuuukas 49506d5
useDepth always if depthOnly is true
jonuuukas 6412f03
fixing useDepth not picking up default depth attachment
jonuuukas 43fc427
adding SupportsNativeRenderPass to rendererFeatures and adding mechan…
jonuuukas a3c0d85
formatting check fix
manuele-bonanno 8357b72
more formatting check fixes...
manuele-bonanno 8a17123
include cleanup
jonuuukas a4ada9d
Merge branch 'master' into universal/render-pass-conversion
jonuuukas c5c3bc0
add validation check and fallback
jonuuukas 00a359a
reviewer feedback changes
jonuuukas 7350695
Merge branch 'master' into universal/render-pass-conversion
jonuuukas 0e287cb
merge cleanup
jonuuukas c606713
minor changes to address reviewers feedback
manuele-bonanno 05a5fa0
minor refactoring addressing reviewer feedback
manuele-bonanno File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
531 changes: 531 additions & 0 deletions
531
com.unity.render-pipelines.universal/Runtime/NativeRenderPass.cs
Large diffs are not rendered by default.
Oops, something went wrong.
3 changes: 3 additions & 0 deletions
3
com.unity.render-pipelines.universal/Runtime/NativeRenderPass.cs.meta
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont see where it is used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lukaschod - yeah, it's not used yet. we can remove it for now, but it will still be added in next PR that will introduce input attachments/fb fetch and etc.