[llvm-commits] CVS: llvm/lib/Transforms/Scalar/SCCP.cpp
Chris Lattner
sabre at nondot.org
Sat Mar 3 20:50:38 PST 2007
Changes in directory llvm/lib/Transforms/Scalar:
SCCP.cpp updated: 1.163 -> 1.164
---
Log message:
minor cleanups
---
Diffs of the changes: (+8 -7)
SCCP.cpp | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
Index: llvm/lib/Transforms/Scalar/SCCP.cpp
diff -u llvm/lib/Transforms/Scalar/SCCP.cpp:1.163 llvm/lib/Transforms/Scalar/SCCP.cpp:1.164
--- llvm/lib/Transforms/Scalar/SCCP.cpp:1.163 Wed Feb 14 20:26:10 2007
+++ llvm/lib/Transforms/Scalar/SCCP.cpp Sat Mar 3 22:50:21 2007
@@ -239,6 +239,9 @@
return TrackedGlobals;
}
+ inline void markOverdefined(Value *V) {
+ markOverdefined(ValueState[V], V);
+ }
private:
// markConstant - Make a value be marked as "constant". If the value
@@ -277,9 +280,6 @@
OverdefinedInstWorkList.push_back(V);
}
}
- inline void markOverdefined(Value *V) {
- markOverdefined(ValueState[V], V);
- }
inline void mergeInValue(LatticeVal &IV, Value *V, LatticeVal &MergeWithV) {
if (IV.isOverdefined() || MergeWithV.isUndefined())
@@ -1365,9 +1365,8 @@
Solver.MarkBlockExecutable(F.begin());
// Mark all arguments to the function as being overdefined.
- std::map<Value*, LatticeVal> &Values = Solver.getValueMapping();
for (Function::arg_iterator AI = F.arg_begin(), E = F.arg_end(); AI != E; ++AI)
- Values[AI].markOverdefined();
+ Solver.markOverdefined(AI);
// Solve for constants.
bool ResolvedUndefs = true;
@@ -1385,6 +1384,8 @@
//
SmallSet<BasicBlock*, 16> &ExecutableBBs = Solver.getExecutableBlocks();
SmallVector<Instruction*, 32> Insts;
+ std::map<Value*, LatticeVal> &Values = Solver.getValueMapping();
+
for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB)
if (!ExecutableBBs.count(BB)) {
DOUT << " BasicBlock Dead:" << *BB;
@@ -1486,14 +1487,13 @@
// Loop over all functions, marking arguments to those with their addresses
// taken or that are external as overdefined.
//
- std::map<Value*, LatticeVal> &Values = Solver.getValueMapping();
for (Module::iterator F = M.begin(), E = M.end(); F != E; ++F)
if (!F->hasInternalLinkage() || AddressIsTaken(F)) {
if (!F->isDeclaration())
Solver.MarkBlockExecutable(F->begin());
for (Function::arg_iterator AI = F->arg_begin(), E = F->arg_end();
AI != E; ++AI)
- Values[AI].markOverdefined();
+ Solver.markOverdefined(AI);
} else {
Solver.AddTrackedFunction(F);
}
@@ -1525,6 +1525,7 @@
SmallSet<BasicBlock*, 16> &ExecutableBBs = Solver.getExecutableBlocks();
SmallVector<Instruction*, 32> Insts;
SmallVector<BasicBlock*, 32> BlocksToErase;
+ std::map<Value*, LatticeVal> &Values = Solver.getValueMapping();
for (Module::iterator F = M.begin(), E = M.end(); F != E; ++F) {
for (Function::arg_iterator AI = F->arg_begin(), E = F->arg_end();
More information about the llvm-commits
mailing list