[PATCH] D16343: [BasicAA] Fix for missing must alias information

Gerolf Hoflehner via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 22:04:46 PST 2016


Gerolf added a comment.

Thank you Duncan + Philip!  Committed NFC is r259290+r259298, the actual fix is 259299. And you convinced me to finally get rid of my outdated C/C++ standards ... :-)


================
Comment at: lib/Analysis/BasicAliasAnalysis.cpp:324
@@ +323,3 @@
+/// particular for 32b programs with negative indices that rely on two's
+/// complement wrap-arounds for correct alias information.
+static int64_t adjustToPointerSize(int64_t Offset, unsigned PointerSize) {
----------------
reames wrote:
> correct -> precise.  Very different in this context.
Done: r259300


http://reviews.llvm.org/D16343





More information about the llvm-commits mailing list