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

[Joel Galenson via Phabricator via llvm-commits]

jgalenson added inline comments.


================
Comment at: lib/Transforms/Scalar/CorrelatedValuePropagation.cpp:364
+
+static void processOverflowIntrinsic(IntrinsicInst *II) {
+  Value *NewOp = nullptr;
----------------
sanjoy wrote:
> `process` is too vague -- how about `eraseOverflowIntrinsic`?
I actually think process is better here because it mirrors all the other functions in this file (processCallSite, processSRem, etc).


https://reviews.llvm.org/D39483