]> granicus.if.org Git - llvm/commitdiff
[NFC] Switch last couple of invariant_load checks to use hasMetadata
authorPhilip Reames <listmail@philipreames.com>
Wed, 4 Sep 2019 18:27:31 +0000 (18:27 +0000)
committerPhilip Reames <listmail@philipreames.com>
Wed, 4 Sep 2019 18:27:31 +0000 (18:27 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@370948 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Analysis/MemorySSA.cpp
lib/Transforms/Scalar/EarlyCSE.cpp
lib/Transforms/Scalar/LICM.cpp
lib/Transforms/Vectorize/LoadStoreVectorizer.cpp

index d41fe15fa077d8f3737763b212d65c86ee241558..ae1e4b80f133419deb6a46c64639757dbb1796da 100644 (file)
@@ -370,7 +370,7 @@ static bool isUseTriviallyOptimizableToLiveOnEntry(AliasAnalysisType &AA,
                                                    const Instruction *I) {
   // If the memory can't be changed, then loads of the memory can't be
   // clobbered.
-  return isa<LoadInst>(I) && (I->getMetadata(LLVMContext::MD_invariant_load) ||
+  return isa<LoadInst>(I) && (I->hasMetadata(LLVMContext::MD_invariant_load) ||
                               AA.pointsToConstantMemory(MemoryLocation(
                                   cast<LoadInst>(I)->getPointerOperand())));
 }
index 09530a6a5aedbe7b6169216eb266f56fcacb78b7..efefaf073ed024d221783bf6b9df6ecedf5d680b 100644 (file)
@@ -791,7 +791,7 @@ bool EarlyCSE::isOperatingOnInvariantMemAt(Instruction *I, unsigned GenAt) {
   // A location loaded from with an invariant_load is assumed to *never* change
   // within the visible scope of the compilation.
   if (auto *LI = dyn_cast<LoadInst>(I))
-    if (LI->getMetadata(LLVMContext::MD_invariant_load))
+    if (LI->hasMetadata(LLVMContext::MD_invariant_load))
       return true;
 
   auto MemLocOpt = MemoryLocation::getOrNone(I);
index 3f3d7f5834e3f75efc4d13c6f8a02d2e501c0ee1..87f027cd5c0f05e198683919c920462fc7c0262e 100644 (file)
@@ -1099,7 +1099,7 @@ bool llvm::canSinkOrHoistInst(Instruction &I, AAResults *AA, DominatorTree *DT,
     // in the same alias set as something that ends up being modified.
     if (AA->pointsToConstantMemory(LI->getOperand(0)))
       return true;
-    if (LI->getMetadata(LLVMContext::MD_invariant_load))
+    if (LI->hasMetadata(LLVMContext::MD_invariant_load))
       return true;
 
     if (LI->isAtomic() && !TargetExecutesOncePerLoop)
index a5b862bf930a8b413339e3143ed3969e11382855..5f97740a9c6f5684c0012e82b61a502818d1d70e 100644 (file)
@@ -665,7 +665,7 @@ Vectorizer::getVectorizablePrefix(ArrayRef<Instruction *> Chain) {
       // We can ignore the alias if the we have a load store pair and the load
       // is known to be invariant. The load cannot be clobbered by the store.
       auto IsInvariantLoad = [](const LoadInst *LI) -> bool {
-        return LI->getMetadata(LLVMContext::MD_invariant_load);
+        return LI->hasMetadata(LLVMContext::MD_invariant_load);
       };
 
       // We can ignore the alias as long as the load comes before the store,