Adding an image load failure case to the shader debug zoo and implementing a fix

baldurk / renderdoc

RenderDoc is a stand-alone graphics debugging tool.

https://renderdoc.org

MIT License

9.07k stars 1.35k forks source link

Adding an image load failure case to the shader debug zoo and implementing a fix #3484

Open Honeybunch opened 1 week ago

Honeybunch commented 1 week ago

Spun off from https://github.com/baldurk/renderdoc/pull/3480

I ran into an assert when trying to debug a SPV shader that performed an imageLoad operation on a buffer texture so I implemented a repro case in the shader debug zoo and what seemed like a reasonable fix to me.

This was specifically when accessing a uint texel buffer with an imageLoad operation. In the case of a float format ReadTexel's default case handling would behave properly.

Honeybunch commented 2 days ago

Sorry this took me so long to get to. I think this approach here is a little more compact 😄