[PATCH] D23798: [Instruction] Introduce a predicate mustOperandBeConstant()

[Sanjay Patel via llvm-commits]

spatel added inline comments.

================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:1383-1385
@@ -1382,5 @@
-                                          unsigned OpIdx) {
-  // Early exit.
-  if (!isa<Constant>(I->getOperand(OpIdx)))
-    return true;
-
----------------
Is this check removed intentionally? Ie, we must assume that the instruction may be malformed?


Repository:
  rL LLVM

https://reviews.llvm.org/D23798