This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Tree inlining for the C front end (part 1 of 3)
- To: Janis Johnson <janis187 at us dot ibm dot com>
- Subject: Re: Tree inlining for the C front end (part 1 of 3)
- From: Alexandre Oliva <aoliva at redhat dot com>
- Date: 10 Oct 2001 04:01:41 -0300
- Cc: gcc-patches at gcc dot gnu dot org
- Organization: GCC Team, Red Hat
- References: <20011009165319.A25711@us.ibm.com>
On Oct 9, 2001, Janis Johnson <janis187@us.ibm.com> wrote:
> The referenced patch results in the following new warnings:
> ../../gcc-mainline/gcc/langhooks.c: In function `tree_inlining_default_hook_copy_res_decl_for_inlining':
> ../../gcc-mainline/gcc/langhooks.c:129: warning: implicit declaration of function `copy_decl_for_inlining'
> ../../gcc-mainline/gcc/langhooks.c:129: warning: return makes pointer from integer without a cast
> Including "insn-config.h" and "integrate.h" in langhooks.c lets the C
> compiler build on ia64-linux. I'm trying a bootstrap and test run now
> and, unless someone comes up with a better patch, will submit this when
> it finishes.
Thanks, I had to include rtl.h too to get this to compile on
athlon-pc-linux.gnu. I'm checking this in, under the
``obviously-correct rule''. Tested on athlon-pc-linux.gnu.
Index: gcc/ChangeLog
from Janis Johnson <janis187@us.ibm.com>, Alexandre Oliva <aoliva@redhat.com>
* langhooks.c: Include rtl.h, insn-config.h and integrate.h.
* Makefile.in (langhooks.o): Add dependencies.
Index: gcc/Makefile.in
===================================================================
RCS file: /cvs/gcc/egcs/gcc/Makefile.in,v
retrieving revision 1.748
diff -u -p -r1.748 Makefile.in
--- gcc/Makefile.in 2001/10/08 20:54:06 1.748
+++ gcc/Makefile.in 2001/10/10 06:58:13
@@ -1349,7 +1349,7 @@ prefix.o: prefix.c $(CONFIG_H) $(SYSTEM_
convert.o: convert.c $(CONFIG_H) $(SYSTEM_H) $(TREE_H) flags.h convert.h toplev.h
langhooks.o : langhooks.c $(CONFIG_H) $(SYSTEM_H) $(TREE_H) toplev.h \
- tree-inline.h
+ tree-inline.h $(RTL_H) insn-config.h integrate.h
tree.o : tree.c $(CONFIG_H) $(SYSTEM_H) $(TREE_H) flags.h function.h toplev.h \
$(GGC_H) $(HASHTAB_H) $(TARGET_H) output.h $(TM_P_H)
tree-inline.o : tree-inline.c $(CONFIG_H) $(SYSTEM_H) $(TREE_H) $(RTL_H) \
Index: gcc/langhooks.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/langhooks.c,v
retrieving revision 1.1
diff -u -p -r1.1 langhooks.c
--- gcc/langhooks.c 2001/10/08 20:54:07 1.1
+++ gcc/langhooks.c 2001/10/10 06:58:13
@@ -24,6 +24,9 @@ Boston, MA 02111-1307, USA. */
#include "toplev.h"
#include "tree.h"
#include "tree-inline.h"
+#include "rtl.h"
+#include "insn-config.h"
+#include "integrate.h"
/* lang_hooks.tree_inlining.walk_subtrees is called by walk_tree()
after handling common cases, but before walking code-specific
--
Alexandre Oliva Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist *Please* write to mailing lists, not to me