[PATCH] D39483: [CVP] Remove some {s|u}{add|sub}.with.overflow checks.

[Joel Galenson via Phabricator via llvm-commits]

jgalenson updated this revision to Diff 121966.
jgalenson added a comment.

Thanks for the comments.  As you suggested, I removed the unnecessary checks.

The new API is indeed cleaner, but it only works for addition.  Modifying it to support other operations might be useful, bit I figured that could be a separate commit if necessary.


https://reviews.llvm.org/D39483

Files:
  lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
  test/Transforms/CorrelatedValuePropagation/overflows.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39483.121966.patch
Type: text/x-patch
Size: 18582 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171107/8adadc92/attachment.bin>