[PATCH] D25135: [InstCombine] sub X, sext(bool Y) -> add X, zext(bool Y)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 13:59:20 PDT 2016


spatel updated this revision to Diff 74579.
spatel added a comment.

Patch updated as suggested by Eli:
Favor the simpler zext+add sequence over keeping an 'nuw' flag.


https://reviews.llvm.org/D25135

Files:
  lib/Transforms/InstCombine/InstCombineAddSub.cpp
  test/Transforms/InstCombine/sub.ll
  test/Transforms/InstCombine/urem.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25135.74579.patch
Type: text/x-patch
Size: 3339 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161013/161936aa/attachment.bin>


More information about the llvm-commits mailing list