[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