[PATCH] D97973: [NFC][AliasSetTracker] Remove implicit conversion AliasResult to integer.

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 4 12:18:55 PST 2021


dfukalov created this revision.
dfukalov added a reviewer: asbirlea.
Herald added a subscriber: hiraditya.
dfukalov requested review of this revision.
Herald added a project: LLVM.

Preparation to make AliasResult scoped enumeration.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D97973

Files:
  llvm/lib/Analysis/AliasSetTracker.cpp


Index: llvm/lib/Analysis/AliasSetTracker.cpp
===================================================================
--- llvm/lib/Analysis/AliasSetTracker.cpp
+++ llvm/lib/Analysis/AliasSetTracker.cpp
@@ -301,7 +301,7 @@
                                                     const AAMDNodes &AAInfo,
                                                     bool &MustAliasAll) {
   AliasSet *FoundSet = nullptr;
-  AliasResult AllAR = MustAlias;
+  MustAliasAll = true;
   for (AliasSet &AS : llvm::make_early_inc_range(*this)) {
     if (AS.Forward)
       continue;
@@ -310,8 +310,8 @@
     if (AR == NoAlias)
       continue;
 
-    AllAR =
-        AliasResult(AllAR & AR); // Possible downgrade to May/Partial, even No
+    if (AR != MustAlias)
+      MustAliasAll = false;
 
     if (!FoundSet) {
       // If this is the first alias set ptr can go into, remember it.
@@ -322,7 +322,6 @@
     }
   }
 
-  MustAliasAll = (AllAR == MustAlias);
   return FoundSet;
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97973.328270.patch
Type: text/x-patch
Size: 964 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210304/c39737f9/attachment.bin>


More information about the llvm-commits mailing list