This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Move decl_default_tls_model prototype to output.h, make more #include rtl.h's redundant
- From: Mark Mitchell <mark at codesourcery dot com>
- To: Steven Bosscher <stevenb dot gcc at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Eric Botcazou <ebotcazou at adacore dot com>
- Date: Tue, 25 May 2010 16:52:29 -0700
- Subject: Re: [patch] Move decl_default_tls_model prototype to output.h, make more #include rtl.h's redundant
- References: <AANLkTinDOWLuPUc8U8Rict6LMe1yS0UkBiKgUkHAvYDB@mail.gmail.com>
Steven Bosscher wrote:
> Is it OK if I consider approval of the middle-end bits
> required but removal of rtl.h and/or expr.h from these front-end files
> as obvious?
Yes.
More generally, removal of header files not in use from any source file
is obvious, not just for you, but for anyone. Of course, people should
be careful not to think about multiple host/target combinations; <foo.h>
may not be required on a GNU/Linux host, but might be required on a
Solaris host and rtl.h might be needed for an ARM target but not a MIPS
target, or some such.
> gcc/ChangeLog:
> * rtl.h (decl_default_tls_model): Move prototype from here...
...
OK.
--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713