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

2007-07-03 Thread Zhou Sheng
Hi Chris, > The condition "IterationCount->getType() != LargestType" doesn't > mean the IterationCount->getType's bitwidth < LargestType's > so, sometimes, (like in this testcase), it need a trunc not ext. > This patch is to fix it. Ah, I see. Thank you for checking in the testcase. The re

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

2007-07-03 Thread Chris Lattner
On Jun 27, 2007, at 7:00 PM, Sheng Zhou wrote: > Chris, > > Attached is the testcase, which will get: > > opt: /developer/home2/zsth/llvm-gcc-dev/HEAD/llvm/llvm/lib/VMCore/ > Constants.cpp:1559: static llvm::Constant* > llvm::ConstantExpr::getZExt(llvm::Constant*, const llvm::Type*): > Asse

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

2007-06-29 Thread Tanya Lattner
Should this be added as a regression test? -Tanya On Jun 27, 2007, at 7:00 PM, Sheng Zhou wrote: > Chris, > > Attached is the testcase, which will get: > > opt: /developer/home2/zsth/llvm-gcc-dev/HEAD/llvm/llvm/lib/VMCore/ > Constants.cpp:1559: static llvm::Constant* > llvm::ConstantExpr::get

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

2007-06-27 Thread Sheng Zhou
Chris, Attached is the testcase, which will get: opt: /developer/home2/zsth/llvm-gcc-dev/HEAD/llvm/llvm/lib/VMCore/Constants.cpp:1559: static llvm::Constant* llvm::ConstantExpr::getZExt(llvm::Constant*, const llvm::Type*): Assertion `C->getType()->getPrimitiveSizeInBits() < Ty->getPrimitiveS

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

2007-06-27 Thread Chris Lattner
On Jun 27, 2007, at 2:50 AM, Zhou Sheng wrote: > Fix a bug. Testcase plz! What does this do, it seems wrong. -Chris > --- > Diffs of the changes: (+4 -1) > > IndVarSimplify.cpp |5 - > 1 files changed, 4 insertions(+), 1 deletion(-) > > > Index: llvm/lib/Transforms/Scalar/IndVarSimpli

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

2007-06-27 Thread Zhou Sheng
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.123 -> 1.124 --- Log message: Fix a bug. --- Diffs of the changes: (+4 -1) IndVarSimplify.cpp |5 - 1 files changed, 4 insertions(+), 1 deletion(-) Index: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp

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

2007-06-19 Thread Dan Gohman
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.122 -> 1.123 LoopStrengthReduce.cpp updated: 1.141 -> 1.142 --- Log message: Rename ScalarEvolution::deleteInstructionFromRecords to deleteValueFromRecords and loosen the types to all it to accept Value* instead of j

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

2007-06-15 Thread Chris Lattner
On Jun 15, 2007, at 7:38 AM, Dan Gohman wrote: > > > Changes in directory llvm/lib/Transforms/Scalar: > > IndVarSimplify.cpp updated: 1.120 -> 1.121 > LoopStrengthReduce.cpp updated: 1.140 -> 1.141 > --- > Log message: > > Add a SCEV class and supporting code for sign-extend expressions. > > This

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

2007-06-15 Thread Dan Gohman
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.121 -> 1.122 --- Log message: Use SCEVConstant::get instead of SCEVUnknown::get to create an integer constant SCEV. --- Diffs of the changes: (+2 -2) IndVarSimplify.cpp |4 ++-- 1 files changed, 2 insertions

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

2007-06-15 Thread Dan Gohman
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.120 -> 1.121 LoopStrengthReduce.cpp updated: 1.140 -> 1.141 --- Log message: Add a SCEV class and supporting code for sign-extend expressions. This created an ambiguity for expandInTy to decide when to use sign-exte

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

2007-06-05 Thread Nick Lewycky
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.119 -> 1.120 --- Log message: Inform ScalarEvolutions that we're deleting Values. This is the obviously correct part of the fix for PR1487: http://llvm.org/PR1487 . --- Diffs of the changes: (+2 -0) IndVarSimpl

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

2007-03-06 Thread Devang Patel
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.114 -> 1.115 --- Log message: Now IndVarSimplify is a LoopPass. --- Diffs of the changes: (+32 -27) IndVarSimplify.cpp | 59 - 1 files changed, 32 insertions

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

2007-03-03 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.113 -> 1.114 --- Log message: make better use of LCSSA information in RewriteLoopExitValues. Before, we would scan the entire loop body, then scan all users of instructions in the loop, looking for users outside the

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

2007-03-03 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.112 -> 1.113 --- Log message: Implement PR1179: http://llvm.org/PR1179 /PR1232: http://llvm.org/PR1232 and test/Transforms/IndVarsSimplify/loop_evaluate_[234].ll This makes -indvars require and use LCSSA, updating

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

2007-03-03 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.111 -> 1.112 --- Log message: Make RewriteLoopExitValues far less nested by using continue in the loop --- Diffs of the changes: (+94 -91) IndVarSimplify.cpp | 185 ++

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp InstructionCombining.cpp LowerGC.cpp

2007-01-31 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.107 -> 1.108 InstructionCombining.cpp updated: 1.617 -> 1.618 LowerGC.cpp updated: 1.19 -> 1.20 --- Log message: remove temporary vectors. --- Diffs of the changes: (+9 -13) IndVarSimplify.cpp |3 +--

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

2007-01-30 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.106 -> 1.107 InstructionCombining.cpp updated: 1.615 -> 1.616 --- Log message: eliminate temporary vectors --- Diffs of the changes: (+13 -8) IndVarSimplify.cpp |5 +++-- InstructionCombining.cpp | 1

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

2007-01-14 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.104 -> 1.105 InstructionCombining.cpp updated: 1.595 -> 1.596 LoopStrengthReduce.cpp updated: 1.105 -> 1.106 ScalarReplAggregates.cpp updated: 1.64 -> 1.65 --- Log message: Eliminate calls to isInteger, generalizing

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

2007-01-12 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.103 -> 1.104 --- Log message: simplify some code --- Diffs of the changes: (+2 -8) IndVarSimplify.cpp | 10 ++ 1 files changed, 2 insertions(+), 8 deletions(-) Index: llvm/lib/Transforms/Scalar/IndVa

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp InstructionCombining.cpp SCCP.cpp ScalarReplAggregates.cpp

2007-01-11 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.102 -> 1.103 InstructionCombining.cpp updated: 1.592 -> 1.593 SCCP.cpp updated: 1.150 -> 1.151 ScalarReplAggregates.cpp updated: 1.63 -> 1.64 --- Log message: For PR1064: http://llvm.org/PR1064 : Implement the arbitr

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

2007-01-08 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.101 -> 1.102 InstructionCombining.cpp updated: 1.586 -> 1.587 ScalarReplAggregates.cpp updated: 1.62 -> 1.63 --- Log message: Comparison of primitive type sizes should now be done in bits, not bytes. This patch conve

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

2007-01-06 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.100 -> 1.101 --- Log message: add -debug output for -indvars. --- Diffs of the changes: (+19 -6) IndVarSimplify.cpp | 25 +++-- 1 files changed, 19 insertions(+), 6 deletions(-) Index: ll

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

2006-09-20 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.94 -> 1.95 --- Log message: Fix Transforms/IndVarsSimplify/2006-09-20-LFTR-Crash.ll --- Diffs of the changes: (+22 -15) IndVarSimplify.cpp | 37 ++--- 1 files changed, 22 insert

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

2006-08-25 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.92 -> 1.93 --- Log message: Add an assertion to check that we're really preserving LCSSA. --- Diffs of the changes: (+2 -0) IndVarSimplify.cpp |2 ++ 1 files changed, 2 insertions(+) Index: llvm/lib/Transf

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

2006-08-25 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.91 -> 1.92 --- Log message: Reapply the indvars patch, since nothing blew up last night. --- Diffs of the changes: (+1 -0) IndVarSimplify.cpp |1 + 1 files changed, 1 insertion(+) Index: llvm/lib/Transform

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

2006-08-24 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.90 -> 1.91 --- Log message: Revert my previous patch. Since there are some major changes that went in today, I'm going to wait to put this in HEAD until tomorrow, so as not to clutter the nightly tester. --- Dif

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

2006-08-24 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.89 -> 1.90 --- Log message: Specify that indvars actually preserve LCSSA. This has been done for a while, but I forgot to put in the analysis usage. --- Diffs of the changes: (+3 -0) IndVarSimplify.cpp |3

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

2006-08-18 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.88 -> 1.89 --- Log message: Fix a grammaro in a comment. --- Diffs of the changes: (+1 -1) IndVarSimplify.cpp |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/lib/Transforms/Scalar/IndVarS

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

2006-07-14 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.87 -> 1.88 --- Log message: Hopefully the final attempt at making IndVars preserve LCSSA. This should fix PR 831: http://llvm.org/PR831 . --- Diffs of the changes: (+30 -2) IndVarSimplify.cpp | 32 +++

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

2006-07-13 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.86 -> 1.87 --- Log message: Revert this patch temporarily until PR831: http://llvm.org/PR831 is fixed. --- Diffs of the changes: (+15 -31) IndVarSimplify.cpp | 46 +++--

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

2006-07-12 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.85 -> 1.86 --- Log message: IndVars now (correctly) preserves LCSSA form. --- Diffs of the changes: (+31 -15) IndVarSimplify.cpp | 46 +++--- 1 files changed, 31 inserti

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

2006-07-11 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.84 -> 1.85 --- Log message: Revert my indvars changes because they were breaking things. Unfortunately this didn't start showing up until after the recent instcombine fixes. --- Diffs of the changes: (+15 -29)

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

2006-06-26 Thread Owen Anderson
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.83 -> 1.84 --- Log message: De-pessimize the handling of LCSSA Phi nodes in IndVarSimplify. Hopefully this will make Shootout-C/nestedloop faster. --- Diffs of the changes: (+29 -15) IndVarSimplify.cpp | 44 +

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

2006-06-16 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.82 -> 1.83 --- Log message: Fix IndVarsSimplify/2006-06-16-Indvar-LCSSA-Crash.ll, a case where a "LCSSA" phi node causes indvars to break dominance properties. This fixes causes indvars to avoid inserting aggressiv

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

2005-11-18 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.81 -> 1.82 --- Log message: Fix a crash building 176.gcc due to my recent patch, which only fixed half the problem. --- Diffs of the changes: (+1 -1) IndVarSimplify.cpp |2 +- 1 files changed, 1 insertion(+)

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

2005-11-17 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: IndVarSimplify.cpp updated: 1.80 -> 1.81 --- Log message: This was checking the wrong GEP expression. Fixing this fixes a gccas crash compiling mysql reported by Ted Kremenek. --- Diffs of the changes: (+1 -1) IndVarSimplify.cpp |2 +-