[llvm-commits] CVS: llvm-test/SingleSource/Benchmarks/Shootout-C++/lists.cpp objinst.cpp

2007-06-07 Thread Evan Cheng
Changes in directory llvm-test/SingleSource/Benchmarks/Shootout-C++: lists.cpp updated: 1.4 -> 1.5 objinst.cpp updated: 1.3 -> 1.4 --- Log message: SMALL_PROBLEM_SIZE --- Diffs of the changes: (+7 -3) lists.cpp |6 +- objinst.cpp |4 ++-- 2 files changed, 7 insertions(+), 3 de

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/GVNPRE.cpp

2007-06-07 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: GVNPRE.cpp updated: 1.23 -> 1.24 --- Log message: Small bugfix, and const-ify some methods (Thanks, Bill). --- Diffs of the changes: (+6 -6) GVNPRE.cpp | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) Index: llvm/lib/T

[llvm-commits] CVS: llvm/lib/Transforms/Utils/LoopSimplify.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Utils: LoopSimplify.cpp updated: 1.99 -> 1.100 --- Log message: Update LoopSimplify to require and preserve DominatorTree only. Now LoopSimplify does not require nor preserve ETForest. --- Diffs of the changes: (+26 -32) LoopSimplify.cpp | 58

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.98 -> 1.99 --- Log message: Update LoopSimplify to require and preserve DominatorTree only. Now LoopSimplify does not require nor preserve ETForest. --- Diffs of the changes: (+4 -0) Dominators.h |4 1 files

[llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/VMCore: Dominators.cpp updated: 1.108 -> 1.109 --- Log message: Update LoopSimplify to require and preserve DominatorTree only. Now LoopSimplify does not require nor preserve ETForest. --- Diffs of the changes: (+12 -3) Dominators.cpp | 15 --- 1

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/GVNPRE.cpp

2007-06-07 Thread Bill Wendling
Hi Owen, One suggestion: > +void dump(std::set& s); > +void dump_unique(std::set& s); My guess is that you want to pass these in as "const". Perhaps even making the method const? -bw ___ llvm-commits mailing list llvm-commits@cs.uiuc.edu http:

[llvm-commits] CVS: llvm/lib/CodeGen/BranchFolding.cpp

2007-06-07 Thread Dale Johannesen
Changes in directory llvm/lib/CodeGen: BranchFolding.cpp updated: 1.66 -> 1.67 --- Log message: Make throttle a hidden parameter, per review. --- Diffs of the changes: (+11 -6) BranchFolding.cpp | 17 +++-- 1 files changed, 11 insertions(+), 6 deletions(-) Index: llvm/lib/C

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/GVNPRE.cpp

2007-06-07 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: GVNPRE.cpp updated: 1.22 -> 1.23 --- Log message: Add partial redundancy elimination. --- Diffs of the changes: (+249 -84) GVNPRE.cpp | 333 + 1 files changed, 249 insertions(+), 8

Re: [llvm-commits] CVS: llvm/lib/CodeGen/BranchFolding.cpp

2007-06-07 Thread Chris Lattner
> BranchFolding.cpp updated: 1.65 -> 1.66 > --- > Log message: > > Throttle tail merging; handling blocks with large numbers of > predecessors > is too slow. Thanks Dale, Please make this a hidden command line option like loop-unswitch- threshold. -Chris > > --- > Diffs of the changes: (+6

[llvm-commits] CVS: llvm/lib/CodeGen/BranchFolding.cpp

2007-06-07 Thread Dale Johannesen
Changes in directory llvm/lib/CodeGen: BranchFolding.cpp updated: 1.65 -> 1.66 --- Log message: Throttle tail merging; handling blocks with large numbers of predecessors is too slow. --- Diffs of the changes: (+6 -2) BranchFolding.cpp |8 ++-- 1 files changed, 6 insertions(+), 2 de

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.97 -> 1.98 --- Log message: Add new method - nearestCommonDominator(). --- Diffs of the changes: (+11 -0) Dominators.h | 11 +++ 1 files changed, 11 insertions(+) Index: llvm/include/llvm/Analysis/Dominato

[llvm-commits] CVS: llvm/include/llvm/Analysis/LoopInfo.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: LoopInfo.h updated: 1.67 -> 1.68 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+3 -3) LoopInfo.h |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) Index: llvm/include/llvm/Analysis/LoopInf

[llvm-commits] CVS: llvm/lib/Analysis/LoopInfo.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Analysis: LoopInfo.cpp updated: 1.86 -> 1.87 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+9 -9) LoopInfo.cpp | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) Index: llvm/lib/Analysis/LoopInfo.

[llvm-commits] CVS: llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Transforms/Utils: BasicBlockUtils.h updated: 1.16 -> 1.17 --- Log message: Do not preserve ETForest. --- Diffs of the changes: (+9 -9) BasicBlockUtils.h | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) Index: llvm/include/

[llvm-commits] CVS: llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Utils: BreakCriticalEdges.cpp updated: 1.52 -> 1.53 --- Log message: Do not preserve ETForest. --- Diffs of the changes: (+5 -26) BreakCriticalEdges.cpp | 31 +-- 1 files changed, 5 insertions(+), 26 deletions(-) Inde

[llvm-commits] CVS: llvm/lib/Analysis/LoadValueNumbering.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Analysis: LoadValueNumbering.cpp updated: 1.42 -> 1.43 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+9 -9) LoadValueNumbering.cpp | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) Index: llvm/li

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.96 -> 1.97 --- Log message: Add instruction level dominates(A,B) interface. --- Diffs of the changes: (+4 -0) Dominators.h |4 1 files changed, 4 insertions(+) Index: llvm/include/llvm/Analysis/Dominators.h

[llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/VMCore: Dominators.cpp updated: 1.107 -> 1.108 --- Log message: Add instruction level dominates(A,B) interface. --- Diffs of the changes: (+23 -0) Dominators.cpp | 23 +++ 1 files changed, 23 insertions(+) Index: llvm/lib/VMCore/Dominat

[llvm-commits] CVS: llvm/lib/CodeGen/IfConversion.cpp

2007-06-07 Thread Evan Cheng
Changes in directory llvm/lib/CodeGen: IfConversion.cpp updated: 1.36 -> 1.37 --- Log message: Only remove the edge from entry to false if false block is merged. --- Diffs of the changes: (+2 -2) IfConversion.cpp |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Index: llvm/li

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/LICM.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: LICM.cpp updated: 1.106 -> 1.107 --- Log message: Do not require ETForest. Now it is unused by LICM. --- Diffs of the changes: (+0 -3) LICM.cpp |3 --- 1 files changed, 3 deletions(-) Index: llvm/lib/Transforms/Scalar/LICM.cpp diff -u

[llvm-commits] CVS: llvm/lib/Transforms/IPO/LoopExtractor.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/IPO: LoopExtractor.cpp updated: 1.28 -> 1.29 --- Log message: Do not use ETForest as well as DomiantorTree. DominatorTree is sufficient. --- Diffs of the changes: (+3 -5) LoopExtractor.cpp |8 +++- 1 files changed, 3 insertions(+), 5 deleti

[llvm-commits] CVS: llvm/lib/Transforms/Utils/CodeExtractor.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Utils: CodeExtractor.cpp updated: 1.56 -> 1.57 --- Log message: Do not use ETForest as well as DomiantorTree. DominatorTree is sufficient. --- Diffs of the changes: (+15 -16) CodeExtractor.cpp | 31 +++ 1 files changed

[llvm-commits] CVS: llvm/include/llvm/Transforms/Utils/FunctionUtils.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Transforms/Utils: FunctionUtils.h updated: 1.11 -> 1.12 --- Log message: Do not use ETForest as well as DomiantorTree. DominatorTree is sufficient. --- Diffs of the changes: (+2 -2) FunctionUtils.h |4 ++-- 1 files changed, 2 insertions(+), 2 del

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.95 -> 1.96 --- Log message: Do not use ETForest as well as DomiantorTree. DominatorTree is sufficient. --- Diffs of the changes: (+10 -0) Dominators.h | 10 ++ 1 files changed, 10 insertions(+) Index: llv

[llvm-commits] CVS: llvm/lib/Transforms/Utils/Mem2Reg.cpp PromoteMemoryToRegister.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Utils: Mem2Reg.cpp updated: 1.30 -> 1.31 PromoteMemoryToRegister.cpp updated: 1.100 -> 1.101 --- Log message: Use DominatorTree instead of ETForest. This allows faster immediate domiantor walk. --- Diffs of the changes: (+15 -13) Mem2Reg.cpp

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/LICM.cpp ScalarReplAggregates.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: LICM.cpp updated: 1.105 -> 1.106 ScalarReplAggregates.cpp updated: 1.96 -> 1.97 --- Log message: Use DominatorTree instead of ETForest. This allows faster immediate domiantor walk. --- Diffs of the changes: (+5 -5) LICM.cpp |

[llvm-commits] CVS: llvm/include/llvm/Transforms/Utils/PromoteMemToReg.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Transforms/Utils: PromoteMemToReg.h updated: 1.12 -> 1.13 --- Log message: Use DominatorTree instead of ETForest. This allows faster immediate domiantor walk. --- Diffs of the changes: (+2 -2) PromoteMemToReg.h |4 ++-- 1 files changed, 2 insertio

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: LoopStrengthReduce.cpp updated: 1.139 -> 1.140 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+7 -8) LoopStrengthReduce.cpp | 15 +++ 1 files changed, 7 insertions(+), 8 deletions(-) Index:

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/CorrelatedExprs.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: CorrelatedExprs.cpp updated: 1.63 -> 1.64 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+25 -23) CorrelatedExprs.cpp | 48 +--- 1 files changed, 25 insertion

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.94 -> 1.95 --- Log message: Add basic block level properlyDominates(A,B) interface. --- Diffs of the changes: (+4 -0) Dominators.h |4 1 files changed, 4 insertions(+) Index: llvm/include/llvm/Analysis/Domi

[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp

2007-06-07 Thread Dale Johannesen
Changes in directory llvm/lib/CodeGen/SelectionDAG: SelectionDAGISel.cpp updated: 1.457 -> 1.458 --- Log message: Do not change the size of function arguments. PR 1489: http://llvm.org/PR1489 . --- Diffs of the changes: (+9 -1) SelectionDAGISel.cpp | 10 +- 1 files changed, 9 in

Re: [llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Nick Lewycky
Devang Patel wrote: > On Jun 7, 2007, at 10:46 AM, Nick Lewycky wrote: >>Similarly, I pass ETNodes around to functions that don't care about >>which BB is actually involved, but do need the DFS numbers for >>whatever reason. > > In that case what's the use of ETNode ? The DFS numbers. ETNode is

Re: [llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Devang Patel
On Jun 7, 2007, at 10:46 AM, Nick Lewycky wrote: > Devang Patel wrote: >> On Jun 6, 2007, at 7:19 PM, Nick Lewycky wrote: >> >>> Predicate simplifier uses this to get the DFS nums for a BB. We also >>> pass around ETNode*s to avoid having a lot of functions doing BB- >>> ETNode >>> >>> looku

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/GCSE.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: GCSE.cpp updated: 1.60 -> 1.61 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+1 -3) GCSE.cpp |4 +--- 1 files changed, 1 insertion(+), 3 deletions(-) Index: llvm/lib/Transforms/Scalar/GCSE.cpp diff

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/GCSE.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: GCSE.cpp updated: 1.59 -> 1.60 --- Log message: Use DominatorTree instead of ETForest. --- Diffs of the changes: (+3 -3) GCSE.cpp |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) Index: llvm/lib/Transforms/Scalar/GCSE.cpp di

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.93 -> 1.94 --- Log message: Add BasicBlock level dominates(A,B) interface. --- Diffs of the changes: (+14 -2) Dominators.h | 16 ++-- 1 files changed, 14 insertions(+), 2 deletions(-) Index: llvm/inclu

[llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/VMCore: Dominators.cpp updated: 1.106 -> 1.107 --- Log message: Maintain ETNode as part of DomTreeNode. This adds redundancy for now. --- Diffs of the changes: (+38 -5) Dominators.cpp | 43 ++- 1 files changed, 38 ins

[llvm-commits] CVS: llvm/lib/Analysis/PostDominators.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Analysis: PostDominators.cpp updated: 1.72 -> 1.73 --- Log message: Maintain ETNode as part of DomTreeNode. This adds redundancy for now. --- Diffs of the changes: (+24 -4) PostDominators.cpp | 28 1 files changed, 24 insertions

[llvm-commits] CVS: llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp LCSSA.cpp

2007-06-07 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Utils: BreakCriticalEdges.cpp updated: 1.51 -> 1.52 LCSSA.cpp updated: 1.45 -> 1.46 --- Log message: Maintain ETNode as part of DomTreeNode. This adds redundancy for now. --- Diffs of the changes: (+2 -2) BreakCriticalEdges.cpp |2 +- LCSSA.cpp

[llvm-commits] CVS: llvm/include/llvm/Analysis/Dominators.h

2007-06-07 Thread Devang Patel
Changes in directory llvm/include/llvm/Analysis: Dominators.h updated: 1.92 -> 1.93 --- Log message: Maintain ETNode as part of DomTreeNode. This adds redundancy for now. --- Diffs of the changes: (+60 -24) Dominators.h | 84 ++- 1

Re: [llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Nick Lewycky
Devang Patel wrote: > On Jun 6, 2007, at 7:19 PM, Nick Lewycky wrote: > >>Predicate simplifier uses this to get the DFS nums for a BB. We also >>pass around ETNode*s to avoid having a lot of functions doing BB- >> >>>ETNode >> >>lookups. > > I am not familiar with predicate simplifier implementa

Re: [llvm-commits] CVS: llvm/lib/VMCore/Dominators.cpp

2007-06-07 Thread Devang Patel
Nick, On Jun 6, 2007, at 7:19 PM, Nick Lewycky wrote: > Predicate simplifier uses this to get the DFS nums for a BB. We also > pass around ETNode*s to avoid having a lot of functions doing BB- > >ETNode > lookups. I am not familiar with predicate simplifier implementation, but why do such loo

Re: [llvm-commits] CVS: llvm/lib/Transforms/IPO/InlineSimple.cpp

2007-06-07 Thread Tanya Lattner
Fixed. Thanks! -Tanya On Jun 7, 2007, at 12:14 AM, Duncan Sands wrote: > Missing spaces after "if": > >> + if(GV == 0) > >> + if(InitList == 0) > >> +if(const Function *f = dyn_cast(Elt)) > > Also, in this last "if" you call the function "f" but I guess > "F" would be more consistent. > >

[llvm-commits] CVS: llvm/lib/Transforms/IPO/InlineSimple.cpp

2007-06-07 Thread Tanya Lattner
Changes in directory llvm/lib/Transforms/IPO: InlineSimple.cpp updated: 1.84 -> 1.85 --- Log message: Formating fixes. --- Diffs of the changes: (+4 -4) InlineSimple.cpp |8 1 files changed, 4 insertions(+), 4 deletions(-) Index: llvm/lib/Transforms/IPO/InlineSimple.cpp diff

[llvm-commits] CVS: llvm/docs/WritingAnLLVMPass.html

2007-06-07 Thread Tanya Lattner
Changes in directory llvm/docs: WritingAnLLVMPass.html updated: 1.61 -> 1.62 --- Log message: Correct typo. Should be "not allowed" --- Diffs of the changes: (+2 -2) WritingAnLLVMPass.html |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Index: llvm/docs/WritingAnLLVMPass.ht

[llvm-commits] CVS: llvm/test/CFrontend/2007-05-07-NestedStructReturn.c

2007-06-07 Thread Duncan Sands
Changes in directory llvm/test/CFrontend: 2007-05-07-NestedStructReturn.c updated: 1.1 -> 1.2 --- Log message: Make the struct bigger, in an attempt to get a "struct return" on more platforms. --- Diffs of the changes: (+1 -1) 2007-05-07-NestedStructReturn.c |2 +- 1 files changed, 1 i

[llvm-commits] CVS: llvm/test/C++Frontend/2007-05-23-TryFinally.cpp

2007-06-07 Thread Duncan Sands
Changes in directory llvm/test/C++Frontend: 2007-05-23-TryFinally.cpp updated: 1.1 -> 1.2 --- Log message: Make this test pass if llvm-g++ was built without exception handling support. --- Diffs of the changes: (+1 -1) 2007-05-23-TryFinally.cpp |2 +- 1 files changed, 1 insertion(+), 1

[llvm-commits] CVS: llvm/lib/CodeGen/IfConversion.cpp

2007-06-07 Thread Evan Cheng
Changes in directory llvm/lib/CodeGen: IfConversion.cpp updated: 1.35 -> 1.36 --- Log message: ifcvt a triangle: don't merge ifcvt block with rejoin block if it can fall through to it. If merged, the resulting block is not a candidate for iterative ifcvting since it contains both predicated a

Re: [llvm-commits] CVS: llvm/lib/Transforms/IPO/InlineSimple.cpp

2007-06-07 Thread Duncan Sands
Missing spaces after "if": > + if(GV == 0) > + if(InitList == 0) > +if(const Function *f = dyn_cast(Elt)) Also, in this last "if" you call the function "f" but I guess "F" would be more consistent. Ciao, Duncan. ___ llvm-commits mailing list l