This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: INSN_SCOPE changes break mips-elf
- From: law at redhat dot com
- To: Jan Hubicka <jh at suse dot cz>
- Cc: Richard Henderson <rth at redhat dot com>, gcc at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Thu, 20 Jun 2002 20:10:20 -0600
- Subject: Re: INSN_SCOPE changes break mips-elf
- Reply-to: law at redhat dot com
In message <20020620190903.GK17171@atrey.karlin.mff.cuni.cz>, Jan Hubicka write
s:
> > On Thu, Jun 20, 2002 at 08:57:08PM +0200, Jan Hubicka wrote:
> > > * cfglayout.c (scope_to_insns_initialize): Call set_block_levels.
> > > (scope_to_insns_finalize): Do not call set_block_levels; handle
> > > sequences.
> > > (choose_inner_scope): New.
> > > * rtl.h (choose_inner_scope): Declare.
> >
> > Yes, this is fine.
>
> Thanks. I've cought compilation error,as call to set_block_levels
> had argument not present in scope_to_insns_initialize and I simply
> didn't waited even long enought to get into the syntactic error during
> build. I am attaching updated patch I hope to commit once testing
> passes.
>
> Thu Jun 20 20:55:19 CEST 2002 Jan Hubicka <jh@suse.cz>
> * cfglayout.c (scope_to_insns_initialize): Call set_block_levels.
> (scope_to_insns_finalize): Do not call set_block_levels; handle
> sequences.
> (choose_inner_scope): New.
> * rtl.h (choose_inner_scope): Declare.
Thanks. I went ahead and installed this after bootstrapping and testing on
hppa2.0w-hp-hpux11.00 and i686-pc-linux-gnu. I also built mips-elf and verified
that debug-6 works.
jeff