workgroupUniformLoad: test pointer-to-atomic variant
See https://github.com/gpuweb/gpuweb/pull/5141
This requires basic testing:
- that all invocations get the same value from the atomic
- that it passes uniformity checks (the load is treated as uniform)
Ideally we would also modify the memory model tests to stress test the case. (That would likely be on me, as I'm the maintainer of those tests)
I think this is done in https://github.com/gpuweb/cts/commit/d3ac5784fb25afe1a9605af57112b854bb8206c7 already?
https://github.com/gpuweb/cts/pull/4332
oh, I didn't now about that!
It's a good start.
- I'd also like to see loading from an atomic inside a larger workgroup variable. The #4332 only tests the case where the atomic is the whole store type to the variable.
- extend uniformity tests (my second point above).
@beaufortfrancois FYI in case you wanted to pursue this further
@dneto0 I'll let you handle it if that's okay. If not, I can start working on it.
@dneto0 I've started https://github.com/gpuweb/cts/pull/4347 and https://github.com/gpuweb/cts/pull/4348
Let me know if we can close this issue or if we shall wait for @dneto0 to modify the memory model tests as well.
Please keep this open for the memory model stress tests. Otherwise I'd open a new one for the stress tests... :-)