[llvm] r263009 - Update comments following the addition of PredicatedScalarEvolution. NFC.
Silviu Baranga via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 9 04:39:06 PST 2016
Author: sbaranga
Date: Wed Mar 9 06:39:06 2016
New Revision: 263009
URL: http://llvm.org/viewvc/llvm-project?rev=263009&view=rev
Log:
Update comments following the addition of PredicatedScalarEvolution. NFC.
We changed several functions in LoopAccessAnalysis to use PSE instead of
taking SE and a SCEV predicate as arguments, but didn't update the comments.
This also fixes a comment in ScalarEvolution, where we refered to Preds
when the argument name was A.
Modified:
llvm/trunk/include/llvm/Analysis/LoopAccessAnalysis.h
llvm/trunk/include/llvm/Analysis/ScalarEvolution.h
Modified: llvm/trunk/include/llvm/Analysis/LoopAccessAnalysis.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/LoopAccessAnalysis.h?rev=263009&r1=263008&r2=263009&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/LoopAccessAnalysis.h (original)
+++ llvm/trunk/include/llvm/Analysis/LoopAccessAnalysis.h Wed Mar 9 06:39:06 2016
@@ -363,10 +363,10 @@ public:
}
/// Insert a pointer and calculate the start and end SCEVs.
- /// \p We need Preds in order to compute the SCEV expression of the pointer
+ /// We need \p PSE in order to compute the SCEV expression of the pointer
/// according to the assumptions that we've made during the analysis.
/// The method might also version the pointer stride according to \p Strides,
- /// and change \p Preds.
+ /// and add new predicates to \p PSE.
void insert(Loop *Lp, Value *Ptr, bool WritePtr, unsigned DepSetId,
unsigned ASId, const ValueToValueMap &Strides,
PredicatedScalarEvolution &PSE);
@@ -638,11 +638,12 @@ private:
Value *stripIntegerCast(Value *V);
-///\brief Return the SCEV corresponding to a pointer with the symbolic stride
-/// replaced with constant one, assuming \p Preds is true.
+/// \brief Return the SCEV corresponding to a pointer with the symbolic stride
+/// replaced with constant one, assuming the SCEV predicate associated with
+/// \p PSE is true.
///
/// If necessary this method will version the stride of the pointer according
-/// to \p PtrToStride and therefore add a new predicate to \p Preds.
+/// to \p PtrToStride and therefore add further predicates to \p PSE.
///
/// If \p OrigPtr is not null, use it to look up the stride value instead of \p
/// Ptr. \p PtrToStride provides the mapping between the pointer value and its
@@ -654,11 +655,11 @@ const SCEV *replaceSymbolicStrideSCEV(Pr
/// \brief If the pointer has a constant stride return it in units of its
/// element size. Otherwise return zero.
///
-/// Ensure that it does not wrap in the address space, assuming \p Preds is
-/// true.
+/// Ensure that it does not wrap in the address space, assuming the predicate
+/// associated with \p PSE is true.
///
/// If necessary this method will version the stride of the pointer according
-/// to \p PtrToStride and therefore add a new predicate to \p Preds.
+/// to \p PtrToStride and therefore add further predicates to \p PSE.
/// The \p Assume parameter indicates if we are allowed to make additional
/// run-time assumptions.
int isStridedPtr(PredicatedScalarEvolution &PSE, Value *Ptr, const Loop *Lp,
Modified: llvm/trunk/include/llvm/Analysis/ScalarEvolution.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/ScalarEvolution.h?rev=263009&r1=263008&r2=263009&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/ScalarEvolution.h (original)
+++ llvm/trunk/include/llvm/Analysis/ScalarEvolution.h Wed Mar 9 06:39:06 2016
@@ -1392,7 +1392,7 @@ namespace llvm {
getWrapPredicate(const SCEVAddRecExpr *AR,
SCEVWrapPredicate::IncrementWrapFlags AddedFlags);
- /// Re-writes the SCEV according to the Predicates in \p Preds.
+ /// Re-writes the SCEV according to the Predicates in \p A.
const SCEV *rewriteUsingPredicate(const SCEV *S, const Loop *L,
SCEVUnionPredicate &A);
/// Tries to convert the \p S expression to an AddRec expression,
More information about the llvm-commits
mailing list