From f6cb4c3d65d8d17f25fd8f19e907efd565cdaf63 Mon Sep 17 00:00:00 2001 From: Daniel Berlin Date: Mon, 24 Sep 2007 22:20:45 +0000 Subject: [PATCH] Comment fixups llvm-svn: 42279 --- include/llvm/ADT/SparseBitVector.h | 1 + lib/Analysis/IPA/Andersens.cpp | 23 +++++++++++------------ 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/include/llvm/ADT/SparseBitVector.h b/include/llvm/ADT/SparseBitVector.h index 97439706c50..fc7ea021603 100644 --- a/include/llvm/ADT/SparseBitVector.h +++ b/include/llvm/ADT/SparseBitVector.h @@ -286,6 +286,7 @@ public: } BecameZero = allzero; } + // Get a hash value for this element; uint64_t getHashValue() const { uint64_t HashVal = 0; diff --git a/lib/Analysis/IPA/Andersens.cpp b/lib/Analysis/IPA/Andersens.cpp index 653ffdded86..247699c6b1e 100644 --- a/lib/Analysis/IPA/Andersens.cpp +++ b/lib/Analysis/IPA/Andersens.cpp @@ -30,11 +30,11 @@ // B can point to. Constraints can handle copies, loads, and stores, and // address taking. // -// The Offline constraint graph optimization portion includes offline variable -// substitution algorithms intended to pointer and location equivalences. -// Pointer equivalences are those pointers that will have the same points-to -// sets, and location equivalences are those variables that always appear -// together in points-to sets. +// The offline constraint graph optimization portion includes offline variable +// substitution algorithms intended to computer pointer and location +// equivalences. Pointer equivalences are those pointers that will have the +// same points-to sets, and location equivalences are those variables that +// always appear together in points-to sets. // // The inclusion constraint solving phase iteratively propagates the inclusion // constraints until a fixed point is reached. This is an O(N^3) algorithm. @@ -137,10 +137,10 @@ namespace { }; // Node class - This class is used to represent a node in the constraint - // graph. Due to various optimizations, not always the case that there is a - // mapping from a Node to a Value. In particular, we add artificial Node's - // that represent the set of pointed-to variables shared for each location - // equivalent Node. + // graph. Due to various optimizations, it is not always the case that + // there is a mapping from a Node to a Value. In particular, we add + // artificial Node's that represent the set of pointed-to variables shared + // for each location equivalent Node. struct Node { Value *Val; SparseBitVector<> *Edges; @@ -160,15 +160,14 @@ namespace { // Number of incoming edges, used during variable substitution to early // free the points-to sets unsigned NumInEdges; - // True if our ponits-to set is in the Set2PEClass map + // True if our points-to set is in the Set2PEClass map bool StoredInHash; - // True if our node has no indirect constraints (Complex or otherwise) + // True if our node has no indirect constraints (complex or otherwise) bool Direct; // True if the node is address taken, *or* it is part of a group of nodes // that must be kept together. This is set to true for functions and // 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. bool AddressTaken; // Nodes in cycles (or in equivalence classes) are united together using a