[PATCH] D139930: [InstCombine] Combine ZExt (B - A) + ZExt(A) to ZExt(B)

luxufan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 15 07:55:42 PST 2022


StephenFan updated this revision to Diff 483191.
StephenFan added a comment.

1. Use new instead of builder.
2. Rename to `commute`
3. Add zext <2 x i5> to <2 x i8> test
4. Add zext i2 to i3 test
5. Add test that has uses.
6. Add negative test of commuted sub operands.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139930

Files:
  llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp
  llvm/test/Transforms/InstCombine/add.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D139930.483191.patch
Type: text/x-patch
Size: 4847 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221215/86117db0/attachment.bin>


More information about the llvm-commits mailing list