[PATCH] D13233: Combining invariant.group metadata bugfix

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 22:25:03 PDT 2015


Prazek created this revision.
Prazek added reviewers: rsmith, majnemer, nlewycky.
Prazek added a subscriber: llvm-commits.

http://reviews.llvm.org/D13233

Files:
  lib/Transforms/Utils/Local.cpp

Index: lib/Transforms/Utils/Local.cpp
===================================================================
--- lib/Transforms/Utils/Local.cpp
+++ lib/Transforms/Utils/Local.cpp
@@ -1426,7 +1426,8 @@
   // then we will just pick it from J - even when they are different.
   // FIXME: we should try to preserve both invariant.group md if they are
   // different, but right now instruction can only have one invariant.group.
-  if (auto *JMD = J->getMetadata(LLVMContext::MD_invariant_group))
+  if (auto *JMD = J->getMetadata(LLVMContext::MD_invariant_group) &&
+      isa<LoadInst>(K))
     K->setMetadata(LLVMContext::MD_invariant_group, JMD);
 
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13233.35937.patch
Type: text/x-patch
Size: 649 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150929/1a5532f8/attachment.bin>


More information about the llvm-commits mailing list