[PATCH] D66608: [InstCombine] icmp eq/ne (gep inbounds P, Idx..), null -> icmp eq/ne P, null

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 20:40:04 PDT 2019


reames updated this revision to Diff 216764.
reames added a comment.

When I went to add the suggested size-0 type bailout, I realized the existing code was correct for this case.  I added tests and a comment to show this.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66608/new/

https://reviews.llvm.org/D66608

Files:
  lib/Transforms/InstCombine/InstCombineCompares.cpp
  test/Transforms/InstCombine/gep-inbounds-null.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66608.216764.patch
Type: text/x-patch
Size: 7575 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190823/b50f088a/attachment.bin>


More information about the llvm-commits mailing list