This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PR 53249: Multiple address modes for same address space

"H.J. Lu" <hjl.tools@gmail.com> writes:
>> Index: gcc/sel-sched-dump.c
>> ===================================================================
>> --- gcc/sel-sched-dump.c    Â2012-05-06 16:17:20.000000000 +0100
>> +++ gcc/sel-sched-dump.c    Â2012-05-06 16:17:20.316206160 +0100
>> @@ -957,7 +957,7 @@ debug_mem_addr_value (rtx x)
>> Â enum machine_mode address_mode;
>>
>> Â gcc_assert (MEM_P (x));
>
> You should remove this assert since  get_address_mode does it.

I think it's better to keep it.

Richard

>> - Âaddress_mode = targetm.addr_space.address_mode (MEM_ADDR_SPACE (x));
>> + Âaddress_mode = get_address_mode (x);
>>
>> Â t = shallow_copy_rtx (x);
>> Â if (cselib_lookup (XEXP (t, 0), address_mode, 0, GET_MODE (t)))
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]