[clang] [llvm] [mlir] [NVPTX] Add support for Shared Cluster Memory address space. (PR #135444)

Alex MacLean via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 19 10:25:34 PDT 2025


================
@@ -426,10 +426,7 @@ static std::optional<bool> evaluateIsSpace(Intrinsic::ID IID, unsigned AS) {
   case Intrinsic::nvvm_isspacep_shared:
     return AS == NVPTXAS::ADDRESS_SPACE_SHARED;
   case Intrinsic::nvvm_isspacep_shared_cluster:
-    // We can't tell shared from shared_cluster at compile time from AS alone,
-    // but it can't be either is AS is not shared.
-    return AS == NVPTXAS::ADDRESS_SPACE_SHARED ? std::nullopt
-                                               : std::optional{false};
+    return AS == NVPTXAS::ADDRESS_SPACE_SHARED_CLUSTER;
----------------
AlexMaclean wrote:

If the address space is `NVPTXAS::ADDRESS_SPACE_SHARED` won't this also return true?

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


More information about the llvm-commits mailing list