[RTL, ColdFire 24/63] Add support for a MODE_INDEX_REG_CLASS macro
Richard Sandiford
richard@codesourcery.com
Wed Feb 28 22:44:00 GMT 2007
Jeffrey Law <law@redhat.com> writes:
>> My concern about efficiency still stands though. Concretely,
>> your patch is going to mean that the MIPS port does (29 modes x
>> 188 regs =) 5452 legitimate address checks at startup, all of
>> which will be false. The figure for PowerPC is (38 modes x 114 regs =)
>> 4332 checks and the figure for ia64 is (40 modes * 334 regs =) 13360 checks.
>> (I realise that ia64 is one of those targets where you expect the compiler
>> to be slower, but still. ;)) As the patch stands, it will also create
>> new garbage addresses for each check. I think at the very least you
>> should:
>>
>> - create one address and use it for all checks, changing parts of
>> it as appropriate
> Definitely. Please go ahead and do this.
Sorry, but I still strongly believe you're leading us in the
wrong direction here. I suggest we leave it for someone else
with less strong feelings. And...
> I was merely showing a quick proof of concept, it wasn't necessarily
> meant to go into the tree as-is, but instead show you the way to
> getting this problem cleanly resolved.
...with respect Jeff, I didn't need you to tell me how the
double_reg_address_ok thing should be implemented. I was resisting
doing it because I disagreed with it, not because I didn't understand
the concept.
Richard
More information about the Gcc-patches
mailing list