[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