[clang] [llvm] [SPIRV][RFC] Rework / extend support for memory scopes (PR #106429)
Dmitry Sidorov via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 9 07:58:33 PDT 2024
MrSidims wrote:
Tagging @svenvh to be aware of the discussion. I personally don't have a strong opinion at the moment of what should be a default. Currently [SPIR-V To LLVM translator](https://github.com/KhronosGroup/SPIRV-LLVM-Translator) picks "Device" as the default (translation of the atomic scope was added in this [PR](https://github.com/KhronosGroup/SPIRV-LLVM-Translator/pull/2222)), it's not necessarily correct, but I do believe that SPIR-V Backend and the translator must be aligned, and if Backend picks another default - the translator's implementation must be adjusted.
https://github.com/llvm/llvm-project/pull/106429
More information about the cfe-commits
mailing list