From: Gil Rapaport <gil.rapaport@intel.com>
Date: Mon, 13 Mar 2017 10:23:46 +0000 (+0000)
Subject: [LV] Set memcheck metadata also for VF==1
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=710c3a271b1b64654f1963f2d4d0ba4dca44cf03;p=llvm

[LV] Set memcheck metadata also for VF==1

This commit is a follow-up on r297580. It fixes the FIXME added temporarily
by that commit to keep the removal of Unroller's specialized version of
scalarizeInstruction() an NFC. See https://reviews.llvm.org/D30715 for details.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@297610 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Transforms/Vectorize/LoopVectorize.cpp b/lib/Transforms/Vectorize/LoopVectorize.cpp
index 50d31563588..43dca02f65f 100644
--- a/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -3167,11 +3167,7 @@ void InnerLoopVectorizer::scalarizeInstruction(Instruction *Instr,
         auto *NewOp = getScalarValue(Instr->getOperand(op), Part, Lane);
         Cloned->setOperand(op, NewOp);
       }
-      // FIXME: Limiting the versioning metadata to VF > 1 is incorrect. It was
-      // added as part of removing Unroller's specialized version of this
-      // method which was not setting versioning metadata.
-      if (VF > 1)
-        addNewMetadata(Cloned, Instr);
+      addNewMetadata(Cloned, Instr);
 
       // Place the cloned scalar in the new loop.
       Builder.Insert(Cloned);