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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> Well, in theory there's a crash on trunk that can happen without exotic pointers (something like "gep i64, i64* %x, i128 INT128_MIN" will cause an assertion failure in getSExtValue().)

Err, wait, nevermind, I misread the patch; that can't happen because it's calling accumulateConstantOffset, not visiting the offsets itself.


https://reviews.llvm.org/D38501