Skip to content

Conversation

@hrydgard
Copy link
Owner

@hrydgard hrydgard commented May 9, 2023

This reverts commit 10dee90, reversing changes made to 34c11c8.

Changes the fog back to be controlled by a fragment shader ID bit instead of by uniforms.

While this did reduce the shader count, I should have carefully benchmarked on older mobile GPUs - while not at all on newer GPUs, on ones like Mali-400, the hit is quite noticeable.

Same with the lmode flag, so I'm gonna have to revert that too. While risky just before a point release, these changes are fairly easy to see if they break..

Then for 1.16 I'll make a "fragment ubershader" flag, so we can choose between max fragment shader performance or minimizing the shader count for less stutter. Or maybe even compile specialized shaders in the background when using the ubershader..

@hrydgard hrydgard added GE emulation Backend-independent GPU issues Performance Unexpected slow performance issues labels May 9, 2023
@hrydgard hrydgard added this to the v1.15.4 milestone May 9, 2023
@hrydgard
Copy link
Owner Author

hrydgard commented May 9, 2023

I'll do a little extra testing before merge btw.

Seems solid.

@hrydgard hrydgard merged commit 38c4c48 into master May 9, 2023
@hrydgard hrydgard deleted the revert-fog-fshader branch May 9, 2023 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GE emulation Backend-independent GPU issues Performance Unexpected slow performance issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants