This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [SH] re-fix the sp_switch attribute
- From: Oleg Endo <oleg dot endo at t-online dot de>
- To: Christian Bruel <christian dot bruel at st dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>, dj at redhat dot com
- Date: Tue, 15 Jan 2013 17:55:28 +0100
- Subject: Re: [SH] re-fix the sp_switch attribute
- References: <50F52C66.3070404@st.com>
Hi,
On Tue, 2013-01-15 at 11:16 +0100, Christian Bruel wrote:
> Hello.
>
> It seems that the .md part from the patch posted in
> http://gcc.gnu.org/ml/gcc-patches/2009-07/msg01797.html
> is missing after rev 150306
>
> Thus compiling the sp_switch attribute documentation example still fail
> with an "unrecognizable insn" error.
>
> However, after re-applying the missing part (updated with new unspec
> enums), the test still fails with a "pcrel too far" assembly error
> because the new stack symbol constant pool entry was not emitted
> (dump_table not called properly)
>
> This patch fixes the sp_switch unspec pattern to be recognized by
> broken_move, where it is now handled to force a dump_table when necessary.
>
> The example from the documentation now compiles fine. Added it to the
> regression tests. OK for trunk ?
>
> ps: Added again the original ChangeLog entry for the missing .md part.
>
I think this is PR 55301, which I opened a while ago.
Christian, when committing, could you please add PR target/55301 to the
ChangeLog etc, so that it pops up in the PR?
Thanks,
Oleg