Merge pull request #31691 from JuliaLang/tb/temp_mdnode

Don't modify existing MDNodes in SIMDloop pass.
JuliaLang · Apr 12, 2019 · 7132c63 · 7132c63
2 parents a84cf6f + 0afa9a3
commit 7132c63
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/llvm-simdloop.cpp b/src/llvm-simdloop.cpp
@@ -203,8 +203,9 @@ bool LowerSIMDLoop::markSIMDLoop(Module &M, Function *marker, bool ivdep)
         MDNode *n = L->getLoopID();
         if (!n) {
             // Loop does not have a LoopID yet, so give it one.
-            n = MDNode::get(Lh->getContext(), ArrayRef<Metadata *>(NULL));
-            n->replaceOperandWith(0, n);
+            auto temp_n = MDNode::getTemporary(Lh->getContext(), ArrayRef<Metadata *>(NULL));
+            temp_n->replaceOperandWith(0, temp_n.get());
+            n = MDNode::replaceWithPermanent(std::move(temp_n));
             L->setLoopID(n);
         }