On Fri, 4 Nov 2011, Torvald Riegel wrote: > 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?
OK with me. -- Joseph S. Myers jos...@codesourcery.com
