[Mlir-commits] [mlir] [MLIR][ROCDL] Fix accessedOperands for Rocdl_RawPtrBufferLoadOp (PR #127983)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Feb 20 02:15:34 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir-llvm

Author: Alexander Weinrauch (AlexAUT)

<details>
<summary>Changes</summary>

`getAccessedOperands` should return all accessed pointer operands. In the case for BufferLoads this is the source buffer and not the result.


---
Full diff: https://github.com/llvm/llvm-project/pull/127983.diff


1 Files Affected:

- (modified) mlir/include/mlir/Dialect/LLVMIR/ROCDLOps.td (+1-1) 


``````````diff
diff --git a/mlir/include/mlir/Dialect/LLVMIR/ROCDLOps.td b/mlir/include/mlir/Dialect/LLVMIR/ROCDLOps.td
index e9dcd112ce54e..fd63745b850fb 100644
--- a/mlir/include/mlir/Dialect/LLVMIR/ROCDLOps.td
+++ b/mlir/include/mlir/Dialect/LLVMIR/ROCDLOps.td
@@ -470,7 +470,7 @@ def ROCDL_RawPtrBufferLoadOp :
   let assemblyFormat = "operands attr-dict `:` type($res)";
   let extraClassDefinition = [{
     ::llvm::SmallVector<::mlir::Value> $cppClass::getAccessedOperands() {
-      return {getRes()};
+      return {getRsrc()};
     }
   }];
 }

``````````

</details>


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


More information about the Mlir-commits mailing list