diff options
| author | Dylan Noblesmith <nobled@dreamwidth.org> | 2014-08-25 00:28:35 +0000 | 
|---|---|---|
| committer | Dylan Noblesmith <nobled@dreamwidth.org> | 2014-08-25 00:28:35 +0000 | 
| commit | d96ce66cb183fab3884eac3929f9c6b2cc66664c (patch) | |
| tree | 5150cc8c50894fd093a31f0c4ac42ecd66b88a03 | |
| parent | 688fa5e15bf4cd9d47621501ad8e79aa93ab1595 (diff) | |
| download | bcm5719-llvm-d96ce66cb183fab3884eac3929f9c6b2cc66664c.tar.gz bcm5719-llvm-d96ce66cb183fab3884eac3929f9c6b2cc66664c.zip  | |
Analysis: take a reference instead of pointer
This parameter is never null.
llvm-svn: 216356
| -rw-r--r-- | llvm/include/llvm/Analysis/DependenceAnalysis.h | 2 | ||||
| -rw-r--r-- | llvm/lib/Analysis/DependenceAnalysis.cpp | 11 | 
2 files changed, 6 insertions, 7 deletions
diff --git a/llvm/include/llvm/Analysis/DependenceAnalysis.h b/llvm/include/llvm/Analysis/DependenceAnalysis.h index 279755e4762..cfe9748a461 100644 --- a/llvm/include/llvm/Analysis/DependenceAnalysis.h +++ b/llvm/include/llvm/Analysis/DependenceAnalysis.h @@ -331,7 +331,7 @@ namespace llvm {      ///      /// breaks the dependence and allows us to vectorize/parallelize      /// both loops. -    const SCEV *getSplitIteration(const Dependence *Dep, unsigned Level); +    const SCEV *getSplitIteration(const Dependence &Dep, unsigned Level);    private:      AliasAnalysis *AA; diff --git a/llvm/lib/Analysis/DependenceAnalysis.cpp b/llvm/lib/Analysis/DependenceAnalysis.cpp index d0784f1e678..312081c676c 100644 --- a/llvm/lib/Analysis/DependenceAnalysis.cpp +++ b/llvm/lib/Analysis/DependenceAnalysis.cpp @@ -168,7 +168,7 @@ void dumpExampleDependence(raw_ostream &OS, Function *F,              for (unsigned Level = 1; Level <= D->getLevels(); Level++) {                if (D->isSplitable(Level)) {                  OS << "da analyze - split level = " << Level; -                OS << ", iteration = " << *DA->getSplitIteration(D, Level); +                OS << ", iteration = " << *DA->getSplitIteration(*D, Level);                  OS << "!\n";                }              } @@ -3729,13 +3729,12 @@ Dependence *DependenceAnalysis::depends(Instruction *Src,  //  // breaks the dependence and allows us to vectorize/parallelize  // both loops. -const  SCEV *DependenceAnalysis::getSplitIteration(const Dependence *Dep, +const  SCEV *DependenceAnalysis::getSplitIteration(const Dependence &Dep,                                                     unsigned SplitLevel) { -  assert(Dep && "expected a pointer to a Dependence"); -  assert(Dep->isSplitable(SplitLevel) && +  assert(Dep.isSplitable(SplitLevel) &&           "Dep should be splitable at SplitLevel"); -  Instruction *Src = Dep->getSrc(); -  Instruction *Dst = Dep->getDst(); +  Instruction *Src = Dep.getSrc(); +  Instruction *Dst = Dep.getDst();    assert(Src->mayReadFromMemory() || Src->mayWriteToMemory());    assert(Dst->mayReadFromMemory() || Dst->mayWriteToMemory());    assert(isLoadOrStore(Src));  | 

