Skip to content
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

Debugger: Add APIs to retrieve render image #11186

Merged
merged 7 commits into from
Jun 17, 2018

Conversation

unknownbrackets
Copy link
Collaborator

This also fixes the debugger using the GLES backend (on desktop GL) to show depth and stencil again.

Under Vulkan, I'm seeing wrong stencil (full for all pixels) for #10634, but haven't verified depth/stencil readback actually works on mobile Vulkan in the general case.

Using PNG encoding with lowest compression settings kept the speed relatively okay. Outside the GE debugger, you could imagine a "trophy app" that takes a screenshot whenever some breakpoint trips. And there's obviously the use case for translation tools.

-[Unknown]

Should be fine to request BGRA->BGRA.

Also, the reverse flags were wrong/ignored on the format.
This allows pulling the display/target while paused.  It may also allow
other actions that work outside a display list.
@unknownbrackets unknownbrackets added this to the v1.7.0 milestone Jun 16, 2018
@unknownbrackets unknownbrackets changed the title Debugger: Add APIs to retrieve render image Debugger: Add APIs to retrieve render image Jun 17, 2018
@hrydgard
Copy link
Owner

Interesting about the bad stencil, I think I've had that working for the GE Debugger in Vulkan but maybe I'm thinking of DX11...

@hrydgard hrydgard merged commit da5f0f7 into hrydgard:master Jun 17, 2018
@unknownbrackets
Copy link
Collaborator Author

Yeah, it works fine in Vulkan on desktop - it's only happening on mobile.

-[Unknown]

@unknownbrackets unknownbrackets deleted the debugger branch June 17, 2018 20:19
@hrydgard
Copy link
Owner

Ah okay, almost certainly driver bugs then, that we should probably report to the vendors..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants