This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [dwarf2cfi] Cleanup interpretation of cfa.reg
- From: Richard Henderson <rth at redhat dot com>
- To: Andreas Schwab <schwab at redhat dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 12 Jul 2011 08:31:06 -0700
- Subject: Re: [dwarf2cfi] Cleanup interpretation of cfa.reg
- References: <4E1B5413.9050702@redhat.com> <m3d3hfj2cn.fsf@hase.home>
On 07/12/2011 02:05 AM, Andreas Schwab wrote:
> Richard Henderson <rth@redhat.com> writes:
>
>> @@ -261,6 +262,15 @@ extern void dwarf2out_set_demangle_name_func (const char *(*) (const char *));
>> extern void dwarf2out_vms_debug_main_pointer (void);
>> #endif
>>
>> +/* Unfortunately, DWARF_FRAME_REGNUM is not universally defined in such a
>> + way as to force an unsigned return type. Do that via inline wrapper. */
>> +
>> +static inline unsigned
>> +dwarf_frame_regnum (unsigned regnum)
>> +{
>> + return DWARF_FRAME_REGNUM (regnum);
>> +}
>> +
>
> I think this has caused the bootstrap failure on ia64:
>
> In file included from ../../gcc/dwarf2cfi.c:31:0:
> ../../gcc/dwarf2out.h: In function 'dwarf_frame_regnum':
> ../../gcc/dwarf2out.h:271:3: error: implicit declaration of function 'ia64_dbx_register_number' [-Werror=implicit-function-declaration]
Dang it. And the whole point of adding that inline was to
enhance portability across targets.
Ok, I'll fix it in a minute.
r~