This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] m68k.h: Add "interrupt_thread" attribute.
- From: Roman Zippel <zippel at linux-m68k dot org>
- To: Andreas Schwab <schwab at suse dot de>
- Cc: Kazu Hirata <kazu at codesourcery dot com>, law at redhat dot com, gcc-patches at gcc dot gnu dot org
- Date: Mon, 14 May 2007 12:56:56 +0200 (CEST)
- Subject: Re: [patch] m68k.h: Add "interrupt_thread" attribute.
- References: <200704271647.l3RGl6mR022901@sparrowhawk.codesourcery.com> <46407ED8.7060801@codesourcery.com> <jemz08ok7n.fsf@sykes.suse.de>
Hi,
On Sun, 13 May 2007, Andreas Schwab wrote:
> > http://gcc.gnu.org/ml/gcc-patches/2007-04/msg01880.html
>
> I think the patch is OK as is.
I still disagree, external API is one thing, but at least the
implementation should be cleaned up.
It would add the following to m68k_compute_frame_layout():
+ bool interrupt_handler = func_kind == m68k_fk_interrupt_handler;
+ bool interrupt_thread = func_kind == m68k_fk_interrupt_thread;
which really belong in m68k_save_reg().
bye, Roman