Respond to comments

- Change getNumOperands to arg size
- Move size check above the for loop

llvm/lib/Target/AMDGPU/AMDGPUOclcReflect.cpp

@@ -61,12 +61,15 @@ PreservedAnalyses AMDGPUOclcReflectPass::run(Function &F,
         !Callee || Callee->getName() != AMDGPU_OCLC_REFLECT)
       continue;
 
-    assert(Call->getNumOperands() == 2 &&
+    assert(Call->arg_size() == 1 &&
            "Wrong number of operands to __oclc_amdgpu_reflect function");
 
     ToRemove.push_back(Call);
   }
 
+  if (!ToRemove.size())
+    return PreservedAnalyses::all();
+
   for (Instruction *I : ToRemove) {
     CallInst *Call = dyn_cast<CallInst>(I);
     const Value *Str = Call->getArgOperand(0);
@@ -83,9 +86,6 @@ PreservedAnalyses AMDGPUOclcReflectPass::run(Function &F,
     I->eraseFromParent();
   }
 
-  if (!ToRemove.size())
-    return PreservedAnalyses::all();
-
   PreservedAnalyses PA;
   PA.preserveSet<CFGAnalyses>();
   PA.preserve<DominatorTreeAnalysis>();