Unconditional Op access

Author: ellishg

llvm/lib/Transforms/Utils/Local.cpp

@@ -4220,19 +4220,19 @@ void llvm::maybeMarkSanitizerLibraryCallNoBuiltin(
 }
 
 bool llvm::canReplaceOperandWithVariable(const Instruction *I, unsigned OpIdx) {
+  const auto *Op = I->getOperand(OpIdx);
   // We can't have a PHI with a metadata type.
-  if (I->getOperand(OpIdx)->getType()->isMetadataTy())
+  if (Op->getType()->isMetadataTy())
     return false;
 
   // swifterror pointers can only be used by a load, store, or as a swifterror
   // argument; swifterror pointers are not allowed to be used in select or phi
   // instructions.
-  if (OpIdx < I->getNumOperands())
-    if (I->getOperand(OpIdx)->isSwiftError())
-      return false;
+  if (Op->isSwiftError())
+    return false;
 
   // Early exit.
-  if (!isa<Constant>(I->getOperand(OpIdx)))
+  if (!isa<Constant>(Op))
     return true;
 
   switch (I->getOpcode()) {