[InlineCost] Fix a couple comments. NFC
authorCraig Topper <craig.topper@intel.com>
Tue, 28 May 2019 07:25:27 +0000 (07:25 +0000)
committerCraig Topper <craig.topper@intel.com>
Tue, 28 May 2019 07:25:27 +0000 (07:25 +0000)
Replace "unary operator" with "unary instruction" in visitUnaryInstruction since
we now have a UnaryOperator class which might needs its own visit function.

Fix a copy/paste in visitCastInst that appears to have been copied from
visitPtrToInt.

llvm-svn: 361794

llvm/lib/Analysis/InlineCost.cpp

index 7fcfc76..ced30d6 100644 (file)
@@ -708,7 +708,7 @@ bool CallAnalyzer::visitIntToPtr(IntToPtrInst &I) {
 }
 
 bool CallAnalyzer::visitCastInst(CastInst &I) {
-  // Propagate constants through ptrtoint.
+  // Propagate constants through casts.
   if (simplifyInstruction(I, [&](SmallVectorImpl<Constant *> &COps) {
         return ConstantExpr::getCast(I.getOpcode(), COps[0], I.getType());
       }))
@@ -744,7 +744,7 @@ bool CallAnalyzer::visitUnaryInstruction(UnaryInstruction &I) {
       }))
     return true;
 
-  // Disable any SROA on the argument to arbitrary unary operators.
+  // Disable any SROA on the argument to arbitrary unary instructions.
   disableSROA(Operand);
 
   return false;