[PATCH] D148986: [InstSimplify] with logical ops: (X | Y) ? 0 : X --> 0

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 23 18:47:24 PDT 2023


Allen updated this revision to Diff 516219.
Allen edited the summary of this revision.
Allen added a comment.

Address comment
1、Use simplifySelectWithICmpEq to handle the implied equalities from the icmp-or according the suggestion
2、Delete the instcombine for test select-cmp-or.ll as it is not need with above refactoring


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

https://reviews.llvm.org/D148986

Files:
  llvm/lib/Analysis/InstructionSimplify.cpp
  llvm/test/Transforms/InstCombine/select-ctlz-to-cttz.ll
  llvm/test/Transforms/InstSimplify/select-cmp-or.ll
  llvm/test/Transforms/InstSimplify/select_or_and.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148986.516219.patch
Type: text/x-patch
Size: 6770 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230424/9e73c2a9/attachment.bin>


More information about the llvm-commits mailing list