[llvm] r272796 - [LV] Add getter function for LoopVectorizationLegality::Strides. NFC

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 15 08:49:46 PDT 2016


Author: anemet
Date: Wed Jun 15 10:49:46 2016
New Revision: 272796

URL: http://llvm.org/viewvc/llvm-project?rev=272796&view=rev
Log:
[LV] Add getter function for LoopVectorizationLegality::Strides. NFC

This should help moving Strides to LAA later.

Modified:
    llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp?rev=272796&r1=272795&r2=272796&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp Wed Jun 15 10:49:46 2016
@@ -1448,6 +1448,8 @@ private:
     emitAnalysisDiag(TheFunction, TheLoop, *Hints, Message);
   }
 
+  ValueToValueMap &getSymbolicStrides() { return SymbolicStrides; }
+
   unsigned NumPredStores;
 
   /// The loop that we evaluate.
@@ -1508,7 +1510,7 @@ private:
   /// Used to emit an analysis of any legality issues.
   LoopVectorizeHints *Hints;
 
-  ValueToValueMap Strides;
+  ValueToValueMap SymbolicStrides;
   SmallPtrSet<Value *, 8> StrideSet;
 
   /// While vectorizing these instructions we have to generate a
@@ -2220,7 +2222,7 @@ int LoopVectorizationLegality::isConsecu
   // We can emit wide load/stores only if the last non-zero index is the
   // induction variable.
   const SCEV *Last = nullptr;
-  if (!Strides.count(Gep))
+  if (!getSymbolicStrides().count(Gep))
     Last = PSE.getSCEV(Gep->getOperand(InductionOperand));
   else {
     // Because of the multiplication by a stride we can have a s/zext cast.
@@ -2232,7 +2234,7 @@ int LoopVectorizationLegality::isConsecu
     //  %idxprom = zext i32 %mul to i64  << Safe cast.
     //  %arrayidx = getelementptr inbounds i32* %B, i64 %idxprom
     //
-    Last = replaceSymbolicStrideSCEV(PSE, Strides,
+    Last = replaceSymbolicStrideSCEV(PSE, getSymbolicStrides(),
                                      Gep->getOperand(InductionOperand), Gep);
     if (const SCEVCastExpr *C = dyn_cast<SCEVCastExpr>(Last))
       Last =
@@ -4663,7 +4665,7 @@ bool LoopVectorizationLegality::canVecto
 
   // Analyze interleaved memory accesses.
   if (UseInterleaved)
-    InterleaveInfo.analyzeInterleaving(Strides);
+    InterleaveInfo.analyzeInterleaving(getSymbolicStrides());
 
   unsigned SCEVThreshold = VectorizeSCEVCheckThreshold;
   if (Hints->getForce() == LoopVectorizeHints::FK_Enabled)
@@ -4952,7 +4954,7 @@ void LoopVectorizationLegality::collectS
 
   DEBUG(dbgs() << "LV: Found a strided access that we can version");
   DEBUG(dbgs() << "  Ptr: " << *Ptr << " Stride: " << *Stride << "\n");
-  Strides[Ptr] = Stride;
+  SymbolicStrides[Ptr] = Stride;
   StrideSet.insert(Stride);
 }
 
@@ -4994,7 +4996,7 @@ void LoopVectorizationLegality::collectL
 }
 
 bool LoopVectorizationLegality::canVectorizeMemory() {
-  LAI = &LAA->getInfo(TheLoop, Strides);
+  LAI = &LAA->getInfo(TheLoop, getSymbolicStrides());
   auto &OptionalReport = LAI->getReport();
   if (OptionalReport)
     emitAnalysis(VectorizationReport(*OptionalReport));




More information about the llvm-commits mailing list