diff mbox

add includes in config/tilepro/mul-tables.c and config/tilegx/mul-tables.c

Message ID CAAgBjM=nvpvg0U6hb+WyBNEUJ=maX6dFa=pD_fa3u3at0V2yYw@mail.gmail.com
State New
Headers show

Commit Message

Prathamesh Kulkarni Jan. 20, 2015, 7:06 a.m. UTC 
Hi,
When I committed r219655, I didn't check in tilepro/mul-tables.c since it
was auto-generated from config/tilepro/gen-mul-tables.cc.
Andrew pointed out to me that this causes the files
confi/tilepro/mul-tables.c and config/tilegx/mul-tables.c to show up
in svn diff. The attached patch fixes that by
putting includes in config/tilepro/mul-tables.c in the order they are
present in
config/tilepro/gen-mul-tables.cc.

The patch won't affect the build since they are generated from
gen-mul-tables.cc, the sole point of this patch is to avoid noise in
svn diff, so could this be
considered similar to a doc-fix ?
OK to commit ?

Thanks,
Prathamesh
2015-01-20  Prathamesh Kulkarni  <prathamesh.kulkarni@linaro.org>

	* config/tilegx/mul-tables.c: Move symtab.h include after coretypes.h include.
	* config/tilepro/mul-tables.c: Add includes hashtab.h, hash-set.h, vec.h,
	machmode.h, tm.h, hard-reg-set.h, input.h, function.h, rtl.h, flags.h, statistics.h,
	double-int.h, real.h, fixed-value.h, alias.h, wide-int.h, inchash.h, tree.h,
	insn-config.h, expmed.h, dojump.h, explow.h, calls.h, emit-rtl.h, varasm.h, stmt.h.

Comments

Richard Biener Jan. 20, 2015, 8:15 a.m. UTC | #1 
On Tue, 20 Jan 2015, Prathamesh Kulkarni wrote:

> Hi,
> When I committed r219655, I didn't check in tilepro/mul-tables.c since it
> was auto-generated from config/tilepro/gen-mul-tables.cc.
> Andrew pointed out to me that this causes the files
> confi/tilepro/mul-tables.c and config/tilegx/mul-tables.c to show up
> in svn diff. The attached patch fixes that by
> putting includes in config/tilepro/mul-tables.c in the order they are
> present in
> config/tilepro/gen-mul-tables.cc.
> 
> The patch won't affect the build since they are generated from
> gen-mul-tables.cc, the sole point of this patch is to avoid noise in
> svn diff, so could this be
> considered similar to a doc-fix ?
> OK to commit ?

Ok.

Thanks,
Richard.
diff mbox

Patch

Index: gcc/config/tilegx/mul-tables.c
===================================================================
--- gcc/config/tilegx/mul-tables.c	(revision 219679)
+++ gcc/config/tilegx/mul-tables.c	(working copy)
@@ -24,6 +24,7 @@ 
 #include "config.h"
 #include "system.h"
 #include "coretypes.h"
+#include "symtab.h"
 #include "hashtab.h"
 #include "hash-set.h"
 #include "vec.h"
@@ -39,7 +40,6 @@ 
 #include "real.h"
 #include "fixed-value.h"
 #include "alias.h"
-#include "symtab.h"
 #include "wide-int.h"
 #include "inchash.h"
 #include "tree.h"
Index: gcc/config/tilepro/mul-tables.c
===================================================================
--- gcc/config/tilepro/mul-tables.c	(revision 219679)
+++ gcc/config/tilepro/mul-tables.c	(working copy)
@@ -24,6 +24,33 @@ 
 #include "config.h"
 #include "system.h"
 #include "coretypes.h"
+#include "symtab.h"
+#include "hashtab.h"
+#include "hash-set.h"
+#include "vec.h"
+#include "machmode.h"
+#include "tm.h"
+#include "hard-reg-set.h"
+#include "input.h"
+#include "function.h"
+#include "rtl.h"
+#include "flags.h"
+#include "statistics.h"
+#include "double-int.h"
+#include "real.h"
+#include "fixed-value.h"
+#include "alias.h"
+#include "wide-int.h"
+#include "inchash.h"
+#include "tree.h"
+#include "insn-config.h"
+#include "expmed.h"
+#include "dojump.h"
+#include "explow.h"
+#include "calls.h"
+#include "emit-rtl.h"
+#include "varasm.h"
+#include "stmt.h"
 #include "expr.h"
 #include "insn-codes.h"
 #include "optabs.h"