-
Notifications
You must be signed in to change notification settings - Fork 781
Remove shader variants from msaa sample #1231
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
Remove shader variants from msaa sample #1231
Conversation
|
|
||
| auto &postprocessing_pass = postprocessing_pipeline->get_pass(0); | ||
| // Select the currently active pipeline | ||
| auto &pipeline = multisampled_depth ? ms_depth_postprocessing_pipeline : postprocessing_pipeline; |
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.
This line is what replaces existing behavior
| outline += depth - getDepth(ivec2(thickness, 0)); | ||
| outline += depth - getDepth(ivec2(0, -thickness)); | ||
|
|
||
| #ifdef OUTLINE_ONLY |
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.
This define is never set anywhere so i removed it
|
For whatever reason, I can't get this PR to compile locally. Maybe you could rebase against main? |
asuessenbach
left a comment
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.
For whatever reason, I don't get that build error anymore!
|
Maybe add |
|
3 approvals - merging |
Description
Contribution towards #1107
Removes shader variant usage from MSAA (other than base shaders)
Keeps existing behavior by creating two pipelines. One with MS Depth enabled and one without
General Checklist:
Please ensure the following points are checked:
Note: The Samples CI runs a number of checks including:
If this PR contains framework changes:
batchcommand line argument to make sure all samples still work properlySample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist: