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

Steven Perron via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 18 07:33:00 PST 2024


================
@@ -3337,11 +3343,8 @@ bool SPIRVInstructionSelector::selectGlobalValue(
   if (HasInit && !Init)
     return true;
 
-  unsigned AddrSpace = GV->getAddressSpace();
-  SPIRV::StorageClass::StorageClass Storage =
-      addressSpaceToStorageClass(AddrSpace, STI);
   bool HasLnkTy = GV->getLinkage() != GlobalValue::InternalLinkage &&
-                  Storage != SPIRV::StorageClass::Function;
+                  StorageClass != SPIRV::StorageClass::Function;
----------------
s-perron wrote:

Can `StorageClass` be `Function` anymore? You change it to `Static` at line 3272, right?
```suggestion
                  StorageClass != SPIRV::StorageClass::Static;
```

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


More information about the llvm-commits mailing list