[Mlir-commits] [mlir] 8039b3f - [MLIR] Fix bad merge with buffer alias analysis.
Frederik Gossen
llvmlistbot at llvm.org
Fri Oct 23 07:11:45 PDT 2020
Author: Frederik Gossen
Date: 2020-10-23T14:11:27Z
New Revision: 8039b3f966804c6578f8265d5eb7bdde8dec3ddd
URL: https://github.com/llvm/llvm-project/commit/8039b3f966804c6578f8265d5eb7bdde8dec3ddd
DIFF: https://github.com/llvm/llvm-project/commit/8039b3f966804c6578f8265d5eb7bdde8dec3ddd.diff
LOG: [MLIR] Fix bad merge with buffer alias analysis.
Added:
Modified:
mlir/lib/Transforms/BufferOptimizations.cpp
Removed:
################################################################################
diff --git a/mlir/lib/Transforms/BufferOptimizations.cpp b/mlir/lib/Transforms/BufferOptimizations.cpp
index cd69a7cc9f13..0a8802e439ec 100644
--- a/mlir/lib/Transforms/BufferOptimizations.cpp
+++ b/mlir/lib/Transforms/BufferOptimizations.cpp
@@ -39,7 +39,7 @@ static bool isSmallAlloc(Value alloc, unsigned maximumSizeInBytes) {
/// Checks whether the given aliases leave the allocation scope.
static bool
leavesAllocationScope(Region *parentRegion,
- const BufferPlacementAliasAnalysis::ValueSetT &aliases) {
+ const BufferAliasAnalysis::ValueSetT &aliases) {
for (Value alias : aliases) {
for (auto *use : alias.getUsers()) {
// If there is at least one alias that leaves the parent region, we know
@@ -54,9 +54,8 @@ leavesAllocationScope(Region *parentRegion,
}
/// Checks, if an automated allocation scope for a given alloc value exists.
-static bool
-hasAllocationScope(Value alloc,
- const BufferPlacementAliasAnalysis &aliasAnalysis) {
+static bool hasAllocationScope(Value alloc,
+ const BufferAliasAnalysis &aliasAnalysis) {
Region *region = alloc.getParentRegion();
do {
if (Operation *parentOp = region->getParentOp()) {
More information about the Mlir-commits
mailing list