This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Do we need to include c-tree.h in varasm.c? (+patch to delete it)
- From: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- To: gcc-patches at gcc dot gnu dot org
- Cc: amodra at bigpond dot net dot au, dje at watson dot ibm dot com, rth at redhat dot com
- Date: Sun, 8 Jun 2003 15:13:28 -0400 (EDT)
- Subject: Do we need to include c-tree.h in varasm.c? (+patch to delete it)
I was curious why we include c-tree.h in varasm.c. I traced this down
to the following patch:
http://gcc.gnu.org/ml/gcc-patches/2002-02/msg01926.html
(I'd like to be able to define something for C/Objc-only in c-tree.h
that the backend files (and other fronends) won't see. This include
of c-tree.h in varasm.c foils that.)
I don't know what if anything c-tree.h was needed for at the time, I
three-staged this on sparc-solaris2.7 which worked, but I can't be
sure something target specific doesn't need it. So I thought I'd ask.
Can I remove it on mainline?
Thanks,
--Kaveh
PS: There's no dependency in Makefile.in to delete.
2003-06-08 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
* varasm.c: Don't include c-tree.h.
diff -rup orig/egcc-CVS20030607/gcc/varasm.c egcc-CVS20030607/gcc/varasm.c
--- orig/egcc-CVS20030607/gcc/varasm.c 2003-06-07 20:01:37.000000000 -0400
+++ egcc-CVS20030607/gcc/varasm.c 2003-06-08 14:20:36.441477000 -0400
@@ -43,7 +43,6 @@ Software Foundation, 59 Temple Place - S
#include "toplev.h"
#include "hashtab.h"
#include "c-pragma.h"
-#include "c-tree.h"
#include "ggc.h"
#include "langhooks.h"
#include "tm_p.h"