[Mlir-commits] [mlir] [AMDGPU] Add parameterization for optimized shared memory variables (PR #82508)

Mehdi Amini llvmlistbot at llvm.org
Mon Feb 26 20:39:13 PST 2024


================
@@ -156,8 +146,9 @@ getShmReadAndWriteOps(Operation *parentOp, Value shmMemRef,
   return success();
 }
 
-LogicalResult amdgpu::optimizeSharedMemoryReadsAndWrites(Operation *parentOp,
-                                                         Value memrefValue) {
+LogicalResult amdgpu::optimizeSharedMemoryReadsAndWrites(
+    Operation *parentOp, Value memrefValue, int64_t kSharedMemoryLineSizeBytes,
+    int64_t kDefaultVectorSizeBits) {
----------------
joker-eph wrote:

Please remove the `k` prefix, it is a Google convention for constants and these aren't constant anymore.

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


More information about the Mlir-commits mailing list