Reducing number of arrays used by reload
Michael Hayes
m.hayes@elec.canterbury.ac.nz
Thu Jan 7 00:57:00 GMT 1999
Jeffrey A Law writes:
> Unfortunately, there's also some indention problems you need to fix
> like:
>
> ! reload_reg_rtx[i] = find_dummy_reload (in, out, inloc, outloc,
> inmode, outmode,
> ! reload_reg_class[i], i,
> earlyclobber_operand_p (out));
OK, I'll clean these up. I was just testing the waters with this
patch to see if it was considred worhtwhile.
> The indention of the arguments needs to be fixed. It sounds like a nit, but
> it is important that we stick to the formatting standards.
>
> Maybe gnu-indent or indent-region within emacs could help?
>
> Can I assume that everything else was a global search and replace + copyright
> updates? I did spot checks only.
Yeah, it was mostly one big regexp search and replace, hence the
indenting was out. I'll fire up indent to fix things up.
I also modified the code to save some of the fields in
choose_reload_regs since a bcopy was no longer applicable. For
clarity, I decided to use the same reload_info structure for the saved
info although it would be trivial to use a simpler structure (say,
save_reload_info) to save a little bit of memory.
Michael.
More information about the Gcc-patches
mailing list