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: [middle-end] Add machine_mode to address_cost target hook


On Tue, Sep 4, 2012 at 12:38 PM, Paolo Bonzini <bonzini@gnu.org> wrote:
> Il 04/09/2012 09:52, Oleg Endo ha scritto:
>> [x] target-independent bits
>> [ ] alpha     [ ] arm       [ ] avr         [ ] bfin
>> [ ] cr16      [ ] cris      [ ] epiphany    [ ] i386
>> [ ] ia64      [ ] iq2000    [ ] lm32        [ ] m32c
>> [ ] m32r      [ ] mcore     [ ] mep         [x] microblaze
>> [x] mips      [ ] mmix      [x] mn10300     [ ] pa
>> [ ] rs6000    [ ] rx        [ ] s390        [ ] score
>> [x] sh        [ ] sparc     [ ] spu         [ ] stormy16
>> [ ] v850      [ ] vax       [ ] xtensa
>>
>> Tested with 'make all-gcc' on SH xgcc and i386 native build.
>> No functional changes, except on MIPS, as requested by Richard
>> Sandiford.
>
> I think you only need explicit approval for mn10300.  All other changes
> are trivial.
>
>> +hook_int_rtx_mode_as_bool_0 (rtx, enum machine_mode, addr_space_t, bool)
>
> So we're using C++ already?  Or do we want ATTRIBUTE_UNUSED here?

Use C++ where it is so nicely obvious an improvement ;)

Richard.

> Paolo


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]