[PATCH] D21512: [InstSimplify] analyze (optionally casted) icmps to eliminate obviously false logic (PR27869)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 12:18:10 PDT 2016


spatel retitled this revision from "[InstSimplify] look through zexts of icmps to eliminate obviously false logic (PR27869)" to "[InstSimplify] analyze (optionally casted) icmps to eliminate obviously false logic (PR27869)".
spatel updated the summary for this revision.
spatel updated this revision to Diff 61284.
spatel added a comment.

Patch updated:

1. Use slightly modified version of David's suggested code to make the transform more general.
2. Add tests to cover more possible cases and some negative cases.
3. Remove the equivalent transform from InstCombine and move its regression tests to InstSimplify.

This now includes what I thought would be the follow-up step in one patch. I can make step #3 a separate commit if that seems safer. Or if I'm wrong in assuming that everything in the InstCombine path is now handled by InstSimplify, please let me know.


http://reviews.llvm.org/D21512

Files:
  lib/Analysis/InstructionSimplify.cpp
  lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
  test/Transforms/InstCombine/and.ll
  test/Transforms/InstCombine/and2.ll
  test/Transforms/InstSimplify/AndOrXor.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21512.61284.patch
Type: text/x-patch
Size: 9298 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160620/be09f9af/attachment.bin>


More information about the llvm-commits mailing list