[patch] 17/n: trans-mem: compiler trans-mem main engine

Torvald Riegel triegel@redhat.com
Fri Nov 4 22:11:00 GMT 2011


On Thu, 2011-11-03 at 20:38 +0000, Joseph S. Myers wrote:
> Make sure that you do need each #include present in this and any other new 
> file.  Since 2008 a lot of includes of tm.h and toplev.h have been removed 
> and diagnostic-core.h introduced as an alternative to diagnostic.h for 
> many users.  If tm.h is needed, it's good to have a comment on the 
> #include explaining why (which target macros are used, for example), since 
> we'd like to avoid unnecessary tm.h includes.

See attached patch. OK for branch?

-------------- next part --------------
commit dddfec5e83627ef4dc208eddad644d478250a1c4
Author: Torvald Riegel <triegel@redhat.com>
Date:   Fri Nov 4 16:59:44 2011 +0100

    Removed unnecessary includes.
    
    	* trans-mem.c: Removed unnecessary includes.

--- a/gcc/ChangeLog.tm
+++ b/gcc/ChangeLog.tm
@@ -1,3 +1,7 @@
+2011-11-04  Torvald Riegel  <triegel@redhat.com>
+
+	* trans-mem.c: Removed unnecessary includes.
+
 2011-11-04  Richard Henderson  <rth@redhat.com>
 
 	* cp/parser.c (cp_parser_init_declarator): Fix production comments.
diff --git a/gcc/trans-mem.c b/gcc/trans-mem.c
index f0f5c8a..db65e12 100644
--- a/gcc/trans-mem.c
+++ b/gcc/trans-mem.c
@@ -20,21 +20,15 @@
 #include "config.h"
 #include "system.h"
 #include "coretypes.h"
-#include "tm.h"
 #include "tree.h"
 #include "gimple.h"
-#include "tree-dump.h"
 #include "tree-flow.h"
 #include "tree-pass.h"
 #include "tree-inline.h"
-#include "except.h"
-#include "diagnostic.h"
-#include "toplev.h"
-#include "flags.h"
+#include "diagnostic-core.h"
 #include "demangle.h"
 #include "output.h"
 #include "trans-mem.h"
-#include "ggc.h"
 #include "params.h"
 #include "target.h"
 #include "langhooks.h"


More information about the Gcc-patches mailing list