This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: New bootstrap failure on ARM systems


> > 
> > The cost on a full 3-stage bootstrap for c/c++/objc/g77/java+building 
> > libraries is about 1.2% compared with the last time I did a full bootstrap 
> > (2003/12/19).  Which isn't especially good news, but isn't too bad (some 
> > of that may be extra java code to compile).
> > 
> > It does at least bootstrap now though.
> > 
> > Native regression tests haven't been run (my machine tripped a kernel bug 
> > again last night), but cross tests on arm-elf have.
> 
> Hi,
> this seems to be the best of all bad sollutions I was able to come with.  I
> moved the crossjumping before if conversion.  This improves code as
> crossjumping seems to make if conversion better.
> Basic block reordering now does just "normal" cleanup that shall not fire that
> often (especially if ifcvt didn't any changes) and I recompute the liveness
> completely for CE targets.
> 
> Does this look acceptable?  On i386 it makes small speedup and it fixes the
> testcase I have.  It should be less expensive than the patch Richard
> tested.
> 
> Honza
> 
> 2004-01-10  Jan Hubicka  <jh@suse.cz>
> 	* toplev.c (HAVE_conditional_execution): Provide default.
> 	(rest_of_handle_reorder_blocks): For conditional_execution target
> 	update liveness once after all transformations
> 	(rest_of_compilation): Do crossjumping before ce3.

This patch bootstraps ok once the uninitialized warning is fixed.  That 
first assignment to 'changed' in rest_of_handle_reorder_blocks should be a 
straight assignment not an or of the uninitialized variable.

R.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]