[Mlir-commits] [mlir] [mlir][analysis] Cleanup collectUnderlyingAddressValues (NFC) (PR #177905)

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: lonely eagle (linuxlonelyeagle)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/177905.diff


1 Files Affected:

- (modified) mlir/lib/Analysis/AliasAnalysis/LocalAliasAnalysis.cpp (+2-4) 


``````````diff
diff --git a/mlir/lib/Analysis/AliasAnalysis/LocalAliasAnalysis.cpp b/mlir/lib/Analysis/AliasAnalysis/LocalAliasAnalysis.cpp
index b698756dd75e9..ff37483c77c8b 100644
--- a/mlir/lib/Analysis/AliasAnalysis/LocalAliasAnalysis.cpp
+++ b/mlir/lib/Analysis/AliasAnalysis/LocalAliasAnalysis.cpp
@@ -170,17 +170,15 @@ static void collectUnderlyingAddressValues(BlockArgument arg, unsigned maxDepth,
     // the entry block.
     SmallVector<RegionSuccessor> successors;
     branch.getSuccessorRegions(RegionBranchPoint::parent(), successors);
-    RegionSuccessor regionSuccessor(region);
-    bool found = false;
+    RegionSuccessor regionSuccessor = RegionSuccessor::parent();
     for (RegionSuccessor &successor : successors) {
       if (successor.getSuccessor() == region) {
         LDBG() << "  Found matching region successor: " << successor;
-        found = true;
         regionSuccessor = successor;
         break;
       }
     }
-    if (!found) {
+    if (regionSuccessor.isParent()) {
       LDBG()
           << "  No matching region successor found, adding argument to output";
       output.push_back(arg);

``````````

</details>


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