Skip to content

Commit 4fc128f

Browse files
[mlir][bufferization][NFC] Clean up code (#78594)
Clean up code and remove dead code.
1 parent 8050359 commit 4fc128f

File tree

3 files changed

+2
-18
lines changed

3 files changed

+2
-18
lines changed

mlir/include/mlir/Dialect/Bufferization/IR/BufferizableOpInterface.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -375,9 +375,6 @@ struct BufferizationOptions {
375375
SmallVector<AnalysisStateInitFn> stateInitializers;
376376
};
377377

378-
/// Return `true` if the given value is a BlockArgument of a func::FuncOp.
379-
bool isFunctionArgument(Value value);
380-
381378
/// Traversal parameters for `findValueInReverseUseDefChain`.
382379
struct TraversalConfig {
383380
/// Specifies if leaves (that do not have further OpOperands to follow)

mlir/lib/Dialect/Bufferization/IR/BufferizableOpInterface.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -759,13 +759,6 @@ LogicalResult BufferizationOptions::createMemCpy(OpBuilder &b, Location loc,
759759
// Bufferization-specific IRMapping support with debugging.
760760
//===----------------------------------------------------------------------===//
761761

762-
bool bufferization::isFunctionArgument(Value value) {
763-
auto bbArg = llvm::dyn_cast<BlockArgument>(value);
764-
if (!bbArg)
765-
return false;
766-
return isa<func::FuncOp>(bbArg.getOwner()->getParentOp());
767-
}
768-
769762
BaseMemRefType bufferization::getMemRefType(Value value,
770763
const BufferizationOptions &options,
771764
MemRefLayoutAttrInterface layout,

mlir/lib/Dialect/Bufferization/Transforms/BufferDeallocationSimplification.cpp

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -240,13 +240,6 @@ struct RemoveRetainedMemrefsGuaranteedToNotAlias
240240
LogicalResult matchAndRewrite(DeallocOp deallocOp,
241241
PatternRewriter &rewriter) const override {
242242
SmallVector<Value> newRetainedMemrefs, replacements;
243-
Value falseValue;
244-
auto getOrCreateFalse = [&]() -> Value {
245-
if (!falseValue)
246-
falseValue = rewriter.create<arith::ConstantOp>(
247-
deallocOp.getLoc(), rewriter.getBoolAttr(false));
248-
return falseValue;
249-
};
250243

251244
for (auto retainedMemref : deallocOp.getRetained()) {
252245
if (potentiallyAliasesMemref(aliasAnalysis, deallocOp.getMemrefs(),
@@ -256,7 +249,8 @@ struct RemoveRetainedMemrefsGuaranteedToNotAlias
256249
continue;
257250
}
258251

259-
replacements.push_back(getOrCreateFalse());
252+
replacements.push_back(rewriter.create<arith::ConstantOp>(
253+
deallocOp.getLoc(), rewriter.getBoolAttr(false)));
260254
}
261255

262256
if (newRetainedMemrefs.size() == deallocOp.getRetained().size())

0 commit comments

Comments
 (0)