From d42ae865b89461bbccdeefbba520c9c1890b3a2d Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Fri, 23 Oct 2015 01:04:45 +0000 Subject: [PATCH] SLPVectorizer: AllSameOpcode* starts "true" only for instructions r251085 wasn't as NFC as intended... From: Mehdi Amini llvm-svn: 251087 --- llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index 0930d580363e..57b09caa5a44 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -2002,9 +2002,6 @@ void BoUpSLP::reorderInputsAccordingToOpcode(ArrayRef VL, SmallVector OrigLeft, OrigRight; - bool AllSameOpcodeLeft = true; - bool AllSameOpcodeRight = true; - if (VL.size()) { // Peel the first iteration out of the loop since there's nothing // interesting to do anyway and it simplifies the checks @@ -2019,6 +2016,10 @@ void BoUpSLP::reorderInputsAccordingToOpcode(ArrayRef VL, Right.push_back(VRight); } + // Keep track if we have instructions with all the same opcode on one side. + bool AllSameOpcodeLeft = isa(Left[0]); + bool AllSameOpcodeRight = isa(Right[0]); + for (unsigned i = 1, e = VL.size(); i != e; ++i) { Instruction *I = cast(VL[i]);