[PATCH] D60061: [InstCombine] ssubo X, C -> saddo X, -C

[Nikita Popov via Phabricator via llvm-commits]

nikic accepted this revision.
nikic added a comment.

LG with nit.



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:2146
 
+    assert(CI.getNumArgOperands() > 1 && "Need at least 2 args for ssubo");
+    Constant *C;
----------------
This assert is a bit odd, because it checks for more than one arg, while ssubo requires exactly two. I think it can just be dropped entirely, we don't seem to have similar asserts in the surrounding code for other intrinsics.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60061/new/

https://reviews.llvm.org/D60061