[llvm-commits] [poolalloc] r119700 - in /poolalloc/trunk/lib/DSA: BottomUpClosure.cpp DSCallGraph.cpp DSTest.cpp
Arushi Aggarwal
aggarwa4 at illinois.edu
Thu Nov 18 08:25:46 PST 2010
Author: aggarwa4
Date: Thu Nov 18 10:25:46 2010
New Revision: 119700
URL: http://llvm.org/viewvc/llvm-project?rev=119700&view=rev
Log:
Formatting changes.
No functionality change.
Modified:
poolalloc/trunk/lib/DSA/BottomUpClosure.cpp
poolalloc/trunk/lib/DSA/DSCallGraph.cpp
poolalloc/trunk/lib/DSA/DSTest.cpp
Modified: poolalloc/trunk/lib/DSA/BottomUpClosure.cpp
URL: http://llvm.org/viewvc/llvm-project/poolalloc/trunk/lib/DSA/BottomUpClosure.cpp?rev=119700&r1=119699&r2=119700&view=diff
==============================================================================
--- poolalloc/trunk/lib/DSA/BottomUpClosure.cpp (original)
+++ poolalloc/trunk/lib/DSA/BottomUpClosure.cpp Thu Nov 18 10:25:46 2010
@@ -733,7 +733,8 @@
// Fast path for noop calls. Note that we don't care about merging globals
// in the callee with nodes in the caller here.
- if (!CS.isIndirectCall() && CS.getRetVal().isNull() && CS.getNumPtrArgs() == 0 && !CS.isVarArg()) {
+ if (!CS.isIndirectCall() && CS.getRetVal().isNull()
+ && CS.getNumPtrArgs() == 0 && !CS.isVarArg()) {
TempFCs.erase(TempFCs.begin());
continue;
}
@@ -755,9 +756,9 @@
// This means, that either it is a direct call site. Or if it is
// an indirect call site, its calleeNode is complete, and we can
// resolve this particular call site.
- assert((CS.isDirectCall() || CS.getCalleeNode()->isCompleteNode())
+ assert((CS.isDirectCall() || CS.getCalleeNode()->isCompleteNode())
&& "Resolving an indirect incomplete call site");
-
+
if (CS.isIndirectCall()) {
++NumIndResolved;
}
@@ -798,13 +799,13 @@
Graph->markIncompleteNodes(DSGraph::MarkFormalArgs);
Graph->computeExternalFlags(DSGraph::DontMarkFormalsExternal);
Graph->computeIntPtrFlags();
-
+
//
// Update the callgraph with the new information that we have gleaned.
// NOTE : This must be called before removeDeadNodes, so that no
// information is lost due to deletion of DSCallNodes.
Graph->buildCallGraph(callgraph,GlobalFunctionList, filterCallees);
-
+
// Delete dead nodes. Treat globals that are unreachable but that can
// reach live nodes as live.
Graph->removeDeadNodes(DSGraph::KeepUnreachableGlobals);
Modified: poolalloc/trunk/lib/DSA/DSCallGraph.cpp
URL: http://llvm.org/viewvc/llvm-project/poolalloc/trunk/lib/DSA/DSCallGraph.cpp?rev=119700&r1=119699&r2=119700&view=diff
==============================================================================
--- poolalloc/trunk/lib/DSA/DSCallGraph.cpp (original)
+++ poolalloc/trunk/lib/DSA/DSCallGraph.cpp Thu Nov 18 10:25:46 2010
@@ -114,11 +114,11 @@
ii != ee; ++ii)
if (!ValMap.count(*ii))
tarjan_rec(*ii, Stack, NextID, ValMap);
-
+
removeECFunctions();
}
-static void removeECs(DSCallGraph::FuncSet& F,
+static void removeECs(DSCallGraph::FuncSet& F,
llvm::EquivalenceClasses<const llvm::Function*>& ECs) {
DSCallGraph::FuncSet result;
for (DSCallGraph::FuncSet::const_iterator ii = F.begin(), ee = F.end();
Modified: poolalloc/trunk/lib/DSA/DSTest.cpp
URL: http://llvm.org/viewvc/llvm-project/poolalloc/trunk/lib/DSA/DSTest.cpp?rev=119700&r1=119699&r2=119700&view=diff
==============================================================================
--- poolalloc/trunk/lib/DSA/DSTest.cpp (original)
+++ poolalloc/trunk/lib/DSA/DSTest.cpp Thu Nov 18 10:25:46 2010
@@ -372,7 +372,8 @@
/// See 'printNode' for more details.
/// Returns true iff the user specified nodes to print.
///
-static bool printNodes(llvm::raw_ostream &O, const Module *M, const DataStructures *DS) {
+static bool printNodes(llvm::raw_ostream &O, const Module *M,
+ const DataStructures *DS) {
cl::list<std::string>::iterator I = PrintNodesForValues.begin(),
E = PrintNodesForValues.end();
if (I != E) {
@@ -391,7 +392,8 @@
/// should be merged, is in fact merged.
/// Returns true iff the user specified any nodes for this option.
///
-static bool checkIfNodesAreSame(llvm::raw_ostream &O, const Module *M, const DataStructures *DS) {
+static bool checkIfNodesAreSame(llvm::raw_ostream &O, const Module *M,
+ const DataStructures *DS) {
// Verify all nodes listed in "CheckNodesSame" belong to the same node.
cl::list<std::string>::iterator I = CheckNodesSame.begin(),
@@ -416,7 +418,8 @@
/// shouldn't be merged, wasn't merged
/// Returns true iff the user specified any nodes for this option.
///
-static bool checkIfNodesAreNotSame(llvm::raw_ostream &O, const Module *M, const DataStructures *DS) {
+static bool checkIfNodesAreNotSame(llvm::raw_ostream &O, const Module *M,
+ const DataStructures *DS) {
// Verify all nodes listed in "CheckNodesNotSame" belong to distinct nodes.
cl::list<std::string>::iterator I = CheckNodesNotSame.begin(),
@@ -450,7 +453,8 @@
/// Returns true iff the user specified anything for this option
///
-static bool checkTypes(llvm::raw_ostream &O, const Module *M, const DataStructures *DS) {
+static bool checkTypes(llvm::raw_ostream &O, const Module *M,
+ const DataStructures *DS) {
// Verify all nodes listed in "CheckType" have the same Type
cl::list<std::string>::iterator I = CheckType.begin(),
@@ -490,7 +494,8 @@
/// Where +flags means 'this node should have these flags'
/// And -flags means 'this node should NOT have these flags'
///
-static bool verifyFlags(llvm::raw_ostream &O, const Module *M, const DataStructures *DS) {
+static bool verifyFlags(llvm::raw_ostream &O, const Module *M,
+ const DataStructures *DS) {
cl::list<std::string>::iterator I = VerifyFlags.begin(),
E = VerifyFlags.end();
if (I != E) {
@@ -549,7 +554,8 @@
///
/// checks that the first function calls the rest of the
/// functions in the list
-static bool checkCallees(llvm::raw_ostream &O, const Module *M, const DataStructures *DS) {
+static bool checkCallees(llvm::raw_ostream &O, const Module *M,
+ const DataStructures *DS) {
//Mangled names must be provided for C++
cl::list<std::string>::iterator I = CheckCallees.begin(),
More information about the llvm-commits
mailing list