Re: PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)

2005-07-21 Thread Richard Henderson
On Tue, Jul 12, 2005 at 12:02:46PM +0200, Laurent GUERBY wrote: > PR tree-optimization/22336 > * function.c (record_block_change): Check for > cfun->ib_boundaries_block. Ok. I don't see that we're going to get anything cleaner for 4.1. r~

Re: PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)

2005-07-12 Thread Zdenek Dvorak
Hello, > I think the patch is fine (although of course I cannot approve it). > > I, as it's author, do not. But, as I keep saying, I don't know what > the proper fix is. preventing record_block_change from working in this situation seems a quite clean solution to me; of course, not expand

Re: PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)

2005-07-12 Thread Richard Kenner
I think the patch is fine (although of course I cannot approve it). I, as it's author, do not. But, as I keep saying, I don't know what the proper fix is. The more proper fix would be to never expand such expressions from ivopts; What are "such expressions"?

Re: PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)

2005-07-12 Thread Zdenek Dvorak
Hello, > If no one is suggesting an alternative, tested on x86 and x86_64-linux > where it restores bootstrap (at last :), ok to commit? I think the patch is fine (although of course I cannot approve it). The more proper fix would be to never expand such expressions from ivopts; I have a patch f

PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)

2005-07-12 Thread Laurent GUERBY
If no one is suggesting an alternative, tested on x86 and x86_64-linux where it restores bootstrap (at last :), ok to commit? We're down to 6 ACATS FAIL on x86_64 and 8 on x86: http://gcc.gnu.org/ml/gcc-testresults/2005-07/msg00654.html http://gcc.gnu.org/ml/gcc-testresults/2005-07/msg00653.html

Re: Problem with tree-ssa-loop-ivopts.c:get_computation-cost

2005-07-05 Thread Laurent GUERBY
This is http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22212 and is the problem blocking Ada bootstrap on x86_64-linux, it would be great to move forward on this. Laurent On Thu, 2005-06-30 at 18:18 -0400, Richard Kenner wrote: > This function generates RTL from an expression to see how many RTL ins

Problem with tree-ssa-loop-ivopts.c:get_computation-cost

2005-06-30 Thread Richard Kenner
This function generates RTL from an expression to see how many RTL insns it is. But this causes a problem compiling the Ada ACATS test cxa4006. The problem is when part of the expression has a location. In that case, record_block_change is called and that relies on cfun->ib_boundaries_block bein