[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