[llvm-commits] [llvm] r167113 - in /llvm/trunk: include/llvm/Analysis/DependenceAnalysis.h lib/Analysis/DependenceAnalysis.cpp

[Preston Briggs]

Looks good to me.

Thanks,
Preston

From: Benjamin Kramer <benny.kra at googlemail.com>
> To: llvm-commits at cs.uiuc.edu
> Subject: [llvm-commits] [llvm] r167113 - in /llvm/trunk:
>         include/llvm/Analysis/DependenceAnalysis.h
>         lib/Analysis/DependenceAnalysis.cpp
> Message-ID: <20121031112533.1E2CE2A6C065 at llvm.org>
> Content-Type: text/plain; charset="utf-8"
>
> Author: d0k
> Date: Wed Oct 31 06:25:32 2012
> New Revision: 167113
>
> URL: http://llvm.org/viewvc/llvm-project?rev=167113&view=rev
> Log:
> Fix a couple of comment typos.
>
> Modified:
>     llvm/trunk/include/llvm/Analysis/DependenceAnalysis.h
>     llvm/trunk/lib/Analysis/DependenceAnalysis.cpp
>
> Modified: llvm/trunk/include/llvm/Analysis/DependenceAnalysis.h
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/DependenceAnalysis.h?rev=167113&r1=167112&r2=167113&view=diff
>
> ==============================================================================
> --- llvm/trunk/include/llvm/Analysis/DependenceAnalysis.h (original)
> +++ llvm/trunk/include/llvm/Analysis/DependenceAnalysis.h Wed Oct 31
> 06:25:32 2012
> @@ -49,7 +49,7 @@
>    /// determine anything beyond the existence of a dependence; that is, it
>    /// represents a confused dependence (see also FullDependence). In most
>    /// cases (for output, flow, and anti dependences), the dependence
> implies
> -  /// an ordering, where the source must preceed the destination; in
> contrast,
> +  /// an ordering, where the source must precede the destination; in
> contrast,
>    /// input dependences are unordered.
>    class Dependence {
>    public:
> @@ -126,7 +126,7 @@
>      virtual bool isConsistent() const { return false; }
>
>      /// getLevels - Returns the number of common loops surrounding the
> -    /// souce and destination of the dependence.
> +    /// source and destination of the dependence.
>      virtual unsigned getLevels() const { return 0; }
>
>      /// getDirection - Returns the direction associated with a particular
> @@ -169,7 +169,7 @@
>    /// able to accurately analyze the interaction of the references; that
> is,
>    /// it is not a confused dependence (see Dependence). In most cases
>    /// (for output, flow, and anti dependences), the dependence implies an
> -  /// ordering, where the source must preceed the destination; in
> contrast,
> +  /// ordering, where the source must precede the destination; in
> contrast,
>    /// input dependences are unordered.
>    class FullDependence : public Dependence {
>    public:
> @@ -195,7 +195,7 @@
>      bool isConsistent() const { return Consistent; }
>
>      /// getLevels - Returns the number of common loops surrounding the
> -    /// souce and destination of the dependence.
> +    /// source and destination of the dependence.
>      unsigned getLevels() const { return Levels; }
>
>      /// getDirection - Returns the direction associated with a particular
> @@ -505,7 +505,7 @@
>
>      /// isKnownPredicate - Compare X and Y using the predicate Pred.
>      /// Basically a wrapper for SCEV::isKnownPredicate,
> -    /// but tries harder, especially in the presense of sign and zero
> +    /// but tries harder, especially in the presence of sign and zero
>      /// extensions and symbolics.
>      bool isKnownPredicate(ICmpInst::Predicate Pred,
>                            const SCEV *X,
> @@ -673,7 +673,7 @@
>      /// where i and j are induction variable, c1 and c2 are loop
> invariant,
>      /// and a and b are constants.
>      /// Returns true if any possible dependence is disproved.
> -    /// Marks the result as inconsistant.
> +    /// Marks the result as inconsistent.
>      /// Works in some cases that symbolicRDIVtest doesn't,
>      /// and vice versa.
>      bool exactRDIVtest(const SCEV *SrcCoeff,
> @@ -689,7 +689,7 @@
>      /// where i and j are induction variable, c1 and c2 are loop
> invariant,
>      /// and a and b are constants.
>      /// Returns true if any possible dependence is disproved.
> -    /// Marks the result as inconsistant.
> +    /// Marks the result as inconsistent.
>      /// Works in some cases that exactRDIVtest doesn't,
>      /// and vice versa. Can also be used as a backup for
>      /// ordinary SIV tests.
> @@ -702,7 +702,7 @@
>
>      /// gcdMIVtest - Tests an MIV subscript pair for dependence.
>      /// Returns true if any possible dependence is disproved.
> -    /// Marks the result as inconsistant.
> +    /// Marks the result as inconsistent.
>      /// Can sometimes disprove the equal direction for 1 or more loops.
>      //  Can handle some symbolics that even the SIV tests don't get,
>      /// so we use it as a backup for everything.
> @@ -712,7 +712,7 @@
>
>      /// banerjeeMIVtest - Tests an MIV subscript pair for dependence.
>      /// Returns true if any possible dependence is disproved.
> -    /// Marks the result as inconsistant.
> +    /// Marks the result as inconsistent.
>      /// Computes directions.
>      bool banerjeeMIVtest(const SCEV *Src,
>                           const SCEV *Dst,
>
> Modified: llvm/trunk/lib/Analysis/DependenceAnalysis.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/DependenceAnalysis.cpp?rev=167113&r1=167112&r2=167113&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Analysis/DependenceAnalysis.cpp (original)
> +++ llvm/trunk/lib/Analysis/DependenceAnalysis.cpp Wed Oct 31 06:25:32 2012
> @@ -1773,7 +1773,7 @@
>  // where i and j are induction variable, c1 and c2 are loop invariant,
>  // and a and b are constants.
>  // Returns true if any possible dependence is disproved.
> -// Marks the result as inconsistant.
> +// Marks the result as inconsistent.
>  // Works in some cases that symbolicRDIVtest doesn't, and vice versa.
>  bool DependenceAnalysis::exactRDIVtest(const SCEV *SrcCoeff,
>                                         const SCEV *DstCoeff,
> @@ -2202,7 +2202,7 @@
>  // gcdMIVtest -
>  // Tests an MIV subscript pair for dependence.
>  // Returns true if any possible dependence is disproved.
> -// Marks the result as inconsistant.
> +// Marks the result as inconsistent.
>  // Can sometimes disprove the equal direction for 1 or more loops,
>  // as discussed in Michael Wolfe's book,
>  // High Performance Compilers for Parallel Computing, page 235.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121031/0186c0bd/attachment.html>