Re: RFA: patch to remove the old RA

[Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>]

Vladimir Makarov <vmakarov@redhat.com> wrote:
> Here is the patch removing the old RA.  The patch removes global.c, 
> local-alloc.c,
> ra-conflict.c fully and big part of regclass.c and regmove.c.  I also 
> renamed
> regclass.c to reginfo.c because the old name wrongly would reflect its 
> contents.

I've tried the patch on powerpc-apple-darwin9.5.0.
The bootstrap failed with

gcc -c  -g -fkeep-inline-functions -DIN_GCC   -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -Wcast-qual -Wold-style-definition -Wmissing-format-attribute -fno-common  -DHAVE_CONFIG_H -I. -I. -I../../LOCAL/trunk/gcc -I../../LOCAL/trunk/gcc/. -I../../LOCAL/trunk/gcc/../include -I./../intl -I../../LOCAL/trunk/gcc/../libcpp/include  -I../../LOCAL/trunk/gcc/../libdecnumber -I../../LOCAL/trunk/gcc/../libdecnumber/dpd -I../libdecnumber    \
		../../LOCAL/trunk/gcc/config/rs6000/rs6000.c -o rs6000.o
In file included from ../../LOCAL/trunk/gcc/config/rs6000/rs6000.c:55:
../../LOCAL/trunk/gcc/gimple.h: In function âgimple_opâ:
../../LOCAL/trunk/gcc/gimple.h:1637: warning: cast discards qualifiers from pointer target type
../../LOCAL/trunk/gcc/gimple.h: In function âgimple_op_ptrâ:
../../LOCAL/trunk/gcc/gimple.h:1651: warning: cast discards qualifiers from pointer target type
../../LOCAL/trunk/gcc/config/rs6000/rs6000.c: In function ârs6000_conditional_register_usageâ:
../../LOCAL/trunk/gcc/config/rs6000/rs6000.c:4605: error: âcall_really_used_regsâ undeclared (first use in this function)
../../LOCAL/trunk/gcc/config/rs6000/rs6000.c:4605: error: (Each undeclared identifier is reported only once
../../LOCAL/trunk/gcc/config/rs6000/rs6000.c:4605: error: for each function it appears in.)

It seems that several backends access call_really_used_reg.
If call_really_used_reg is backed to global from static,
then the build fails with

gcc  -g -fkeep-inline-functions -DIN_GCC   -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -Wcast-qual -Wold-style-definition -Wmissing-format-attribute -fno-common  -DHAVE_CONFIG_H  -o cc1-dummy c-lang.o stub-objc.o attribs.o c-errors.o c-lex.o c-pragma.o c-decl.o c-typeck.o c-convert.o c-aux-info.o c-common.o c-opts.o c-format.o c-semantics.o c-ppoutput.o c-cppbuiltin.o c-objc-common.o c-dump.o c-pch.o c-parser.o darwin-c.o rs6000-c.o c-gimplify.o tree-mudflap.o c-pretty-print.o c-omp.o dummy-checksum.o \
	  main.o tree-browser.o libbackend.a ../libcpp/libcpp.a ../libdecnumber/libdecnumber.a ../libcpp/libcpp.a ./../intl/libintl.a -liconv  ../libiberty/libiberty.a ../libdecnumber/libdecnumber.a -lmpfr -lgmp  
Undefined symbols:
  "_try_auto_increment", referenced from:
      _fixup_match_2 in libbackend.a(regmove.o)
      _fixup_match_2 in libbackend.a(regmove.o)

Regards,
	kaz