alan-baker
alan-baker
I forgot about validation. Minor updates are necessary there.
> Is there a plan to move forward with this PR? > > I am working on a tool to verify SPIRV wrt the vulkan memory model and having clspv...
Should think how this interacts with #4956. Would the samplers all be required to be filterable or not?
That certainly seems reasonable. I think this is the first instance (for WGSL at least) of dependent features.
I agree that it would be nice to get to that point, but while the proposal lists subgroups-f16 it makes sense to make it dependent. We haven't really discussed removing...
> These tests LGTM but maybe we should also add some coverage via let in an execution test somewhere (like your `textureDimensions` change)? Otherwise for Tint we won't hit the...
If `M` is declared as `static` the problem goes away. clspv does not seem to be handling the non-static case correctly. Presumably LLVM cannot remove the variable since it expects...
The styling of the spec doesn't really draw the eye to table names (in light mode at least) and that's the only place the stage and direction are listed. I'm...
https://github.com/llvm/llvm-project/commit/4d60c6d9b2c863d773aac9b59af8780e5ba23fcd
I do think compat is a special case because it is adding restrictions instead of relaxations should I wouldn't advocate using it as precedence for other decisions. I don't think...