[PATCH] D38862: Add must alias info to ModRefInfo.

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 23 02:58:15 PST 2017


nlopes added inline comments.


================
Comment at: lib/Analysis/AliasAnalysis.cpp:506
+    if (AR == MustAlias)
+      return MRI_MustModRef;
+  }
----------------
nlopes wrote:
> AtomicCmpXchgInst must read, but may not write (depending on the value read).
> I think the only safe value here is MRI_ModRef, since the current lattice doesn't have a more precise value for MustRead & MayReadWithMustAlias.
gah, I meant MustRef & MayModWithMustAlias.


https://reviews.llvm.org/D38862





More information about the llvm-commits mailing list