Ian Lance Taylor wrote:
Jeff Law <l...@redhat.com> writes:

I would fix this in LEGITIMIZE_RELOAD_ADDRESS or in
TARGET_SECONDARY_RELOAD.  I don't know why cc1 crashed, you will have to
debug that.
LEGITIMIZE_RELOAD_ADDRESS is not the right place to handle this -- LEGITIMIZE_RELOAD_ADDRESS is to be used when target specific
approaches for reloading can generate more efficient code than the
generic code in reload.   The generic code should be producing
correct, though potentially inefficient code.

I will just note that this is a long-standing point of disagreement
between Jeff and me.
True. But in this specific case we're talking about an out of range offset that requires a specific register for the secondary reload. That's a case reload should already be handling correctly.
Ian

Reply via email to