Re: [reload] paradoxical mems

2006-01-16 Thread DJ Delorie
> Patch is OK if bootstrapped, etc. Passed bootstrap/check/regression. Committed. Thanks! 2006-01-16 DJ Delorie <[EMAIL PROTECTED]> * reload.c (reg_overlap_mentioned_for_reload_p): Handle subregs of mems. Index: reload.c =

Re: [reload] paradoxical mems

2006-01-13 Thread Ian Lance Taylor
DJ Delorie <[EMAIL PROTECTED]> writes: > > Reload can temporarily produce a SUBREG of a MEM if, as the comment > > says, the insn has a SUBREG of a REG, and the REG is turned into a > > memory address. But before approving this patch, can you say > > something about what calls reg_overlap_mention

Re: [reload] paradoxical mems

2006-01-13 Thread DJ Delorie
> Reload can temporarily produce a SUBREG of a MEM if, as the comment > says, the insn has a SUBREG of a REG, and the REG is turned into a > memory address. But before approving this patch, can you say > something about what calls reg_overlap_mentioned_for_reload_p in this > case? There are a nu

Re: [reload] paradoxical mems

2006-01-12 Thread Ian Lance Taylor
DJ Delorie <[EMAIL PROTECTED]> writes: > reg_overlap_mentioned_for_reload_p() assumes that all SUBREGs > have REGs as their operand. However, register_operand() has > this comment: > >(Ideally, (SUBREG (MEM)...) should not exist after reload, >but currently it does result from (S

[reload] paradoxical mems

2006-01-12 Thread DJ Delorie
reg_overlap_mentioned_for_reload_p() assumes that all SUBREGs have REGs as their operand. However, register_operand() has this comment: (Ideally, (SUBREG (MEM)...) should not exist after reload, but currently it does result from (SUBREG (REG)...) where the reg went on