[PATCH] D32686: [InstCombine][KnownBits] Use KnownBits better to detect nsw adds

[Yoav Ben-Shalom via Phabricator via llvm-commits]

yabash updated this revision to Diff 97265.
yabash added a comment.

Thanks for the comments, fixed.

I opted for std::move and not += because both don't allocate,
and std::move looks more readable (dedicated name 'Result').
Let me know what you think.


https://reviews.llvm.org/D32686

Files:
  lib/Transforms/InstCombine/InstCombineAddSub.cpp
  test/Transforms/InstCombine/AddOverFlow.ll
  test/Transforms/InstCombine/demand_shrink_nsw.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32686.97265.patch
Type: text/x-patch
Size: 6838 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170501/214ca192/attachment.bin>