[PATCH] D51929: [DAGCombiner] use UADDO to optimize saturated unsigned add

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 12 16:30:01 PDT 2018


spatel updated this revision to Diff 165176.
spatel marked 2 inline comments as done.
spatel added a comment.

Patch updated:

1. Removed NFC formatting (https://reviews.llvm.org/rL342095).
2. Added IR example to code comment. This transform has a lot of predicates, and there's no direct twin in IR, so I'm not sure this actually makes it better...but let me know. :)


https://reviews.llvm.org/D51929

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/AArch64/sat-add.ll
  test/CodeGen/X86/sat-add.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51929.165176.patch
Type: text/x-patch
Size: 5249 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180912/16b21729/attachment.bin>


More information about the llvm-commits mailing list