[PATCH] D38501: [ValueTracking] Fix a misuse of APInt in GetPointerBaseWithConstantOffset

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 7 21:04:47 PDT 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D38501#890491, @mppf wrote:

> @hfinkel - I've made the change you requested. I think this one is good to go, but as I mentioned we need to commit it after another fix to BasicAA (such as https://reviews.llvm.org/D38499), or else the new test will fail with BasicAA assertions.


LGTM


https://reviews.llvm.org/D38501





More information about the llvm-commits mailing list