This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Unreviewed Patch [H8/300] : Patch for -mno-exr option in caseof function with "monitor" attribute
- From: Kazu Hirata <kazu at cs dot umass dot edu>
- To: SherryS at KPITCummins dot com
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 14 Jul 2004 20:01:46 -0400 (EDT)
- Subject: Re: Unreviewed Patch [H8/300] : Patch for -mno-exr option in caseof function with "monitor" attribute
- References: <4A1BE23A7B777442B60F4B4916AE0F13022B51B9@sohm.kpit.com>
Hi Sherry,
> The monitor function can accept parameters and return values. The
> monitor attribute requires the
> "h8300_current_function_monitor_function_p()" to be declared.
Why? Neither the current source code nor your patch has a call to
h8300_current_function_monitor_function_p. How is it called? I am
wondering if you have missed a hunk or two when you submitted your
patch.
> If this is not done the monitor function will be treated similar to
> interrupt function which cannot accept parameters or return values.
> The patch http://gcc.gnu.org/ml/gcc-patches/2004-06/msg02495.html
> implements the behavior similar to IAR compiler. We have done the
> debugging in native gdb and it was found that it uses the
> "h8300_current_function_monitor_function_p()" whenever the monitor
> attribute is used.
Are you defining h8300_current_function_monitor_function_p so that you
can call it from gdb?
Kazu Hirata