[jit] Use standard initial includes

David Malcolm dmalcolm@redhat.com
Wed Sep 24 16:39:00 GMT 2014


On Tue, 2014-09-23 at 23:27 +0000, Joseph S. Myers wrote:
[...]
> > +#include "config.h"
> > +#include "system.h"
> > +#include "ansidecl.h"
> > +#include "coretypes.h"
> 
> The standard initial includes are config.h, system.h, coretypes.h.  
> system.h includes libiberty.h which includes ansidecl.h, so direct 
> ansidecl.h includes shouldn't be needed anywhere.
[...]

I've committed the following fix for the above to branch dmalcolm/jit:

gcc/jit/ChangeLog.jit:

	* dummy-frontend.c: Update copyright year.  Follow standard for
	initial includes by removing redundant include of "ansidecl.h".
	* internal-api.c: Follow standard for initial includes by removing
	redundant include of "ansidecl.h".
	* jit-builtins.c: Likewise.
	* libgccjit.c: Likewise.
---
 gcc/jit/ChangeLog.jit    | 9 +++++++++
 gcc/jit/dummy-frontend.c | 3 +--
 gcc/jit/internal-api.c   | 1 -
 gcc/jit/jit-builtins.c   | 1 -
 gcc/jit/libgccjit.c      | 1 -
 5 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/gcc/jit/ChangeLog.jit b/gcc/jit/ChangeLog.jit
index f451771..4ddd3cb 100644
--- a/gcc/jit/ChangeLog.jit
+++ b/gcc/jit/ChangeLog.jit
@@ -1,5 +1,14 @@
 2014-09-24  David Malcolm  <dmalcolm@redhat.com>
 
+	* dummy-frontend.c: Update copyright year.  Follow standard for
+	initial includes by removing redundant include of "ansidecl.h".
+	* internal-api.c: Follow standard for initial includes by removing
+	redundant include of "ansidecl.h".
+	* jit-builtins.c: Likewise.
+	* libgccjit.c: Likewise.
+
+2014-09-24  David Malcolm  <dmalcolm@redhat.com>
+
 	* ChangeLog.jit: Add copyright footer.
 	* Make-lang.in: Update copyright.
 	* config-lang.in: Update copyright.
diff --git a/gcc/jit/dummy-frontend.c b/gcc/jit/dummy-frontend.c
index 1b96c91..1d178f9 100644
--- a/gcc/jit/dummy-frontend.c
+++ b/gcc/jit/dummy-frontend.c
@@ -1,5 +1,5 @@
 /* jit.c -- Dummy "frontend" for use during JIT-compilation.
-   Copyright (C) 2013 Free Software Foundation, Inc.
+   Copyright (C) 2013-2014 Free Software Foundation, Inc.
 
 This file is part of GCC.
 
@@ -19,7 +19,6 @@ along with GCC; see the file COPYING3.  If not see
 
 #include "config.h"
 #include "system.h"
-#include "ansidecl.h"
 #include "coretypes.h"
 #include "opts.h"
 #include "signop.h"
diff --git a/gcc/jit/internal-api.c b/gcc/jit/internal-api.c
index 9e59d92..76ada70 100644
--- a/gcc/jit/internal-api.c
+++ b/gcc/jit/internal-api.c
@@ -20,7 +20,6 @@ along with GCC; see the file COPYING3.  If not see
 
 #include "config.h"
 #include "system.h"
-#include "ansidecl.h"
 #include "coretypes.h"
 #include "opts.h"
 #include "tree.h"
diff --git a/gcc/jit/jit-builtins.c b/gcc/jit/jit-builtins.c
index 160ef20..c4b0f59 100644
--- a/gcc/jit/jit-builtins.c
+++ b/gcc/jit/jit-builtins.c
@@ -19,7 +19,6 @@ along with GCC; see the file COPYING3.  If not see
 
 #include "config.h"
 #include "system.h"
-#include "ansidecl.h"
 #include "coretypes.h"
 #include "opts.h"
 #include "tree.h"
diff --git a/gcc/jit/libgccjit.c b/gcc/jit/libgccjit.c
index 510ed86..cb8321c 100644
--- a/gcc/jit/libgccjit.c
+++ b/gcc/jit/libgccjit.c
@@ -20,7 +20,6 @@ along with GCC; see the file COPYING3.  If not see
 
 #include "config.h"
 #include "system.h"
-#include "ansidecl.h"
 #include "coretypes.h"
 #include "opts.h"
 
-- 
1.7.11.7



More information about the Gcc-patches mailing list