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 rtl-optimization/15248 -- semi-latent reload bug


Joern RENNECKE wrote:

The comment needs updating.

Yes, I'll fix that in the next iteration.
> ! /* If this register is being made equivalent to a MEM
> ! and the MEM is not SET_SRC, the equivalencing insn
> ! is one with the MEM as a SET_DEST and it occurs later.
> ! So don't mark this insn now. */

The original comment was probably true when considering execution time, but not
necessarily when looking at the static instructions.

It should be, since local-alloc only considers registers that occur in a single basic block for this kind of equivalence.


Anyhow, with the help of Eric Botcazou I've found out what's wrong - it doesn't correctly handle equivalences made for parameter registers correctly in all cases. I'm currently testing a new patch, which is unfortunately a bit larger than the previous one.


Bernd



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