s-perron wrote: > @s-perron the addition of --target-env vulkan1.3 creates an issue right now > where we cannot use SequentiallyConsistent due to the new vulkan memory > model. The barriers do not work with that memory model currently anyway. Is > there a way to change the memory model for the test OR can we move to another > memory semantic (AcquireRelease?) OR can I remove the --target-env flag from > the test?
The issue is that you are not allowed to use SequentiallyConsistent in Vulkan. See the [spec](https://docs.vulkan.org/spec/latest/appendices/spirvenv.html#VUID-StandaloneSpirv-MemorySemantics-10866). You can look up the VUID. You should be generating the same code that DXC generates for these: https://godbolt.org/z/3PMKcKPqn You want to use `AcquireRelease` with the appropriate memory type. In this case `WorkgroupMemory`. https://github.com/llvm/llvm-project/pull/185383 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
