RFA: 4.2 reload infrastructure to fix PR target/21623

Kaz Kojima kkojima@rr.iij4u.or.jp
Fri Oct 7 23:47:00 GMT 2005


Joern RENNECKE <joern.rennecke@st.com> wrote:
> Before deciding on how to best install an interim patch for 4.1, I would 
> like
> us to agree on the interface we are going to use for 4.2 - at least 
> inasmuch as
> it would matter for portability of code from the interim patch to the 
> permanent
> solution.

I'm trying to test your patch on my machines.  I've found that a tiny
change is needed for powerpc-apple-darwin7.9.0 because the pcc backend
has its own secondary_reload_class function defined in the rs6000
files.  I've renamed it to rs6000_secondary_reload_class in my test.
It looks that alpha, mn10300, pa and rs6000 define their own
secondary_reload_class.  BTW, I've got a compiler warning during build

../gcc/gcc/reload1.c: In function `emit_reload_insns':
../gcc/gcc/reload1.c:6541: warning: 'letter' might be used uninitialized in this function

and it seems that "letter = *constraint;" is needed just before the line
6552 "new_class = (letter == 'r' ? GENERAL_REGS".

Regards,
	kaz



More information about the Gcc-patches mailing list