[PATCH] D38499: [BasicAA] Fix adjustToPointerSize in BasicAliasAnalysis.cpp for ptr > 64b
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 3 14:08:53 PDT 2017
efriedma added a comment.
If we switch to using APInt for offsets, adjustToPointerSize() just goes away, so I don't see how this is forward progress.
I guess the alternative is to try to add a bunch of overflow checks, but that seems a lot more complicated for no benefit.
https://reviews.llvm.org/D38499
More information about the llvm-commits
mailing list