[PATCH] D64533: [IndVars] Special case the problematic (gep inbounds p, iv == nullptr) problem (pr42357)
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 23 11:10:06 PDT 2019
reames abandoned this revision.
reames added a comment.
Alternate change landed, so patch abandoned.
In D64533#1641871 <https://reviews.llvm.org/D64533#1641871>, @jdoerfert wrote:
> FWIW, I like the D66608 <https://reviews.llvm.org/D66608> approach way better.
For the record, it was the discussion on the PoisonChecking review which made me realize the alternate approach was possible. Particularly the comments about the out of bounds base immediately triggering poison for an inbounds GEP. Without that, the alternate approach didn't seem feasible. So thanks!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64533/new/
https://reviews.llvm.org/D64533
More information about the llvm-commits
mailing list