This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: New bootstrap failure on ARM systems
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Jan Hubicka <jh at suse dot cz>
- Cc: Richard dot Earnshaw at arm dot com, gcc-patches at gcc dot gnu dot org, Jan Hubicka <hubicka at ucw dot cz>
- Date: Mon, 12 Jan 2004 10:32:23 +0000
- Subject: Re: New bootstrap failure on ARM systems
- Organization: ARM Ltd.
- Reply-to: Richard dot Earnshaw at arm dot com
> >
> > 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.