[Mlir-commits] [mlir] c2246eb - [MLIR][NFC] Remove unused argument in affine scalrep helper util

Uday Bondhugula llvmlistbot at llvm.org
Wed Feb 9 18:56:26 PST 2022


Author: Uday Bondhugula
Date: 2022-02-10T08:25:55+05:30
New Revision: c2246eb893c1cedbd9dedccca87764c82697485a

URL: https://github.com/llvm/llvm-project/commit/c2246eb893c1cedbd9dedccca87764c82697485a
DIFF: https://github.com/llvm/llvm-project/commit/c2246eb893c1cedbd9dedccca87764c82697485a.diff

LOG: [MLIR][NFC] Remove unused argument in affine scalrep helper util

NFC. Remove unused argument in affine scalrep helper utility.

Differential Revision: https://reviews.llvm.org/D119397

Added: 
    

Modified: 
    mlir/lib/Dialect/Affine/Utils/Utils.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/Affine/Utils/Utils.cpp b/mlir/lib/Dialect/Affine/Utils/Utils.cpp
index 1e3a38f51cb61..f6acfeadb9852 100644
--- a/mlir/lib/Dialect/Affine/Utils/Utils.cpp
+++ b/mlir/lib/Dialect/Affine/Utils/Utils.cpp
@@ -684,7 +684,6 @@ static LogicalResult forwardStoreToLoad(
 // 3) There is no potential read between writeA and writeB.
 static void findUnusedStore(AffineWriteOpInterface writeA,
                             SmallVectorImpl<Operation *> &opsToErase,
-                            SmallPtrSetImpl<Value> &memrefsToErase,
                             PostDominanceInfo &postDominanceInfo) {
 
   for (Operation *user : writeA.getMemRef().getUsers()) {
@@ -831,7 +830,7 @@ void mlir::affineScalarReplace(FuncOp f, DominanceInfo &domInfo,
 
   // Walk all store's and perform unused store elimination
   f.walk([&](AffineWriteOpInterface storeOp) {
-    findUnusedStore(storeOp, opsToErase, memrefsToErase, postDomInfo);
+    findUnusedStore(storeOp, opsToErase, postDomInfo);
   });
   // Erase all store op's which don't impact the program
   for (auto *op : opsToErase)


        


More information about the Mlir-commits mailing list