From 710c3a271b1b64654f1963f2d4d0ba4dca44cf03 Mon Sep 17 00:00:00 2001 From: Gil Rapaport Date: Mon, 13 Mar 2017 10:23:46 +0000 Subject: [PATCH] [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 --- lib/Transforms/Vectorize/LoopVectorize.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) 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); -- 2.50.1