[llvm] [SPIR-V] Fixup storage class for global private (PR #116636)

[Dmitry Sidorov via llvm-commits]

Nathan =?utf-8?q?Gauër?= <brioche at google.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/116636 at github.com>


================
@@ -205,6 +205,10 @@ addressSpaceToStorageClass(unsigned AddrSpace, const SPIRVSubtarget &STI) {
                : SPIRV::StorageClass::CrossWorkgroup;
   case 7:
     return SPIRV::StorageClass::Input;
+  case 8:
+    return SPIRV::StorageClass::Output;
+  case 9:
+    return SPIRV::StorageClass::Private;
----------------
MrSidims wrote:

There are some potential synchronization issues between the repositories that we should address in the future. Would you mind to move Private to address space 10? The reason I'm asking for this is that the [llvm-spirv translator uses it for CodeSectionINTEL storage class](https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/88b18319c943e70c2475b495806226e117fdaa15/lib/SPIRV/SPIRVInternal.h#L192) :)

https://github.com/llvm/llvm-project/pull/116636