Inline stripInvariantGroupMetadata out of existence
authorSanjoy Das <sanjoy@playingwithpointers.com>
Fri, 16 Dec 2016 20:29:39 +0000 (20:29 +0000)
committerSanjoy Das <sanjoy@playingwithpointers.com>
Fri, 16 Dec 2016 20:29:39 +0000 (20:29 +0000)
As a one liner function, I don't think it is pulling its weight in terms
of helping readability.

llvm-svn: 289987

llvm/lib/CodeGen/CodeGenPrepare.cpp

index bc333b9..0ff0993 100644 (file)
@@ -206,7 +206,6 @@ class TypePromotionTransaction;
                         unsigned CreatedInstCost);
     bool splitBranchCondition(Function &F);
     bool simplifyOffsetableRelocate(Instruction &I);
-    void stripInvariantGroupMetadata(Instruction &I);
   };
 }
 
@@ -5316,7 +5315,7 @@ bool CodeGenPrepare::optimizeInst(Instruction *I, bool& ModifiedDT) {
       return OptimizeCmpExpression(CI, TLI);
 
   if (LoadInst *LI = dyn_cast<LoadInst>(I)) {
-    stripInvariantGroupMetadata(*LI);
+    LI->setMetadata(LLVMContext::MD_invariant_group, nullptr);
     if (TLI) {
       bool Modified = optimizeLoadExt(LI);
       unsigned AS = LI->getPointerAddressSpace();
@@ -5327,7 +5326,7 @@ bool CodeGenPrepare::optimizeInst(Instruction *I, bool& ModifiedDT) {
   }
 
   if (StoreInst *SI = dyn_cast<StoreInst>(I)) {
-    stripInvariantGroupMetadata(*SI);
+    SI->setMetadata(LLVMContext::MD_invariant_group, nullptr);
     if (TLI) {
       unsigned AS = SI->getPointerAddressSpace();
       return optimizeMemoryInst(I, SI->getOperand(1),
@@ -5733,7 +5732,3 @@ bool CodeGenPrepare::splitBranchCondition(Function &F) {
   }
   return MadeChange;
 }
-
-void CodeGenPrepare::stripInvariantGroupMetadata(Instruction &I) {
-  I.setMetadata(LLVMContext::MD_invariant_group, nullptr);
-}