[RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno
David Edelsohn
dje.gcc@gmail.com
Sat Aug 2 01:46:00 GMT 2014
On Fri, Aug 1, 2014 at 2:03 PM, rohitarulraj@freescale.com
<rohitarulraj@freescale.com> wrote:
> Hello Ulrich,
>
> Thanks.
>
>> > /* Use gcc hard register numbering for eh_frame. */ -#define
>> >DWARF_FRAME_REGNUM(REGNO) (REGNO)
>> >+#define DWARF_FRAME_REGNUM(REGNO) \
>> >+ ((REGNO) >= FIRST_SPE_HIGH_REGNO ? ((REGNO) -
>> FIRST_SPE_HIGH_REGNO +
>> >+1200) : (REGNO))
>>
>> Any reason for not using SPE_HIGH_REGNO_P here, just in case we do get
>> other hard registers at some point?
>
> Yes, we can use it. I just have to move the definition of "SPE_HIGH_REGNO_P" macro before "DWARF_FRAME_REGNUM" macro definition.
> [Previously, I had defined and placed "SPE_HIGH_REGNO_P" macro along with similar macros "ALTIVEC_REGNO_P" etc.]
>
> I had updated the patch as required (For this last change, I have checked/tested only the builds: ppc64 trunk, e500v2 v4.9.1 bareboard & linux build).
>
> PR target/60102
>
> [libgcc]
> 2014-07-31 Rohit <rohitarulraj@freescale.com>
> * config/rs6000/linux-unwind.h (ppc_fallback_frame_state): Update
> based on change in SPE high register numbers and 3 HTM registers.
>
> [gcc]
> 2014-07-31 Rohit <rohitarulraj@freescale.com>
> * config/rs6000/rs6000.c
> (rs6000_reg_names) : Add SPE high register names.
> (alt_reg_names) : Likewise.
> (rs6000_dwarf_register_span) : For SPE high registers, replace
> dwarf register numbers with GCC hard register numbers.
> (rs6000_init_dwarf_reg_sizes_extra) : Likewise.
> (rs6000_dbx_register_number): For SPE high registers, return dwarf
> register number for the corresponding GCC hard register number.
>
> * config/rs6000/rs6000.h
> (FIRST_PSEUDO_REGISTER) : Update based on 32 newly added GCC hard
> register numbers for SPE high registers.
> (DWARF_FRAME_REGISTERS) : Likewise.
> (DWARF_REG_TO_UNWIND_COLUMN) : Likewise.
> (DWARF_FRAME_REGNUM) : Likewise.
> (FIXED_REGISTERS) : Likewise.
> (CALL_USED_REGISTERS) : Likewise.
> (CALL_REALLY_USED_REGISTERS) : Likewise.
> (REG_ALLOC_ORDER) : Likewise.
> (enum reg_class) : Likewise.
> (REG_CLASS_NAMES) : Likewise.
> (REG_CLASS_CONTENTS) : Likewise.
> (SPE_HIGH_REGNO_P) : New macro to identify SPE high registers.
>
> * gcc.target/powerpc/pr60102.c: New testcase.
The patch is okay with me if Uli is satisfied.
Thanks, David
More information about the Gcc-patches
mailing list