On 9/24/07, Duncan Sands <[EMAIL PROTECTED]> wrote:
> Hi DannyB,
>
> > @@ -287,6 +286,14 @@
> > }
> > BecameZero = allzero;
> > }
> > + // Get a hash value for this element;
> > + uint64_t getHashValue() const {
>
> shouldn't there be a blank line before the added lines?
>
> > +// The Offline constraint graph optimization portion includes offline
> > variable
>
> The Offline -> The offline
>
> > +// substitution algorithms intended to pointer and location equivalences.
>
> Does not parse!
>
> > + // graph. Due to various optimizations, not always the case that
> > there is a
>
> not always the case -> it is not always the case
>
> > + // True if our ponits-to set is in the Set2PEClass map
>
> ponits-to -> points-to
>
> > + // True if our node has no indirect constraints (Complex or
> > otherwise)
>
> Complex -> complex
>
> > + // their arg nodes, which must be kept at the same position relative
> > to
> > + // their base function node.
> > + // kept at the same position relative to their base function node.
>
> Looks like this last line shouldn't be there.
All fixed
>
> Ciao,
>
> Duncan.
>
_______________________________________________
llvm-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
