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

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 9 16:10:23 PST 2017


sanjoy accepted this revision.
sanjoy added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Transforms/Scalar/CorrelatedValuePropagation.cpp:359
+  default:
+    llvm_unreachable("Illegal instruction.");
+  case Intrinsic::uadd_with_overflow:
----------------
Super-minor nit:  s/Illegal/Unexpected/


https://reviews.llvm.org/D39483





More information about the llvm-commits mailing list