Patchwork [gcc47] Deprecate -fconserve-space

login
register
mail settings
Submitter Steven Bosscher
Date June 4, 2012, 8:30 p.m.
Message ID <CABu31nONkhYdtRxQ6Ko0R-w+T1_XdZe+HZEcaoE-rYNdAcZUVQ@mail.gmail.com>
Download mbox | patch
Permalink /patch/162871/
State New
Headers show

Comments

Steven Bosscher - June 4, 2012, 8:30 p.m.
On Mon, Jun 4, 2012 at 9:45 PM, Jason Merrill <jason@redhat.com> wrote:
> In wwwdocs I would also mention that the flag has no effect on most targets.

How about this:


?

>  In fact, I'm not sure it affects any targets that we still actually
> support; are there any a.out targets left?

There is still a file config/arm/aout.h, and there appears to be code
in config/cris/cris.c to support an unnamed non-ELF target that may be
a.out. But it's hard to tell what other targets are doing.

Ciao!
Steven
Jason Merrill - June 4, 2012, 8:40 p.m.
On 06/04/2012 04:30 PM, Steven Bosscher wrote:
> How about this:

OK, looks good.

> There is still a file config/arm/aout.h

I noticed that, but it seems to be used together with elfos.h, so it 
seems not to be a.out-specific.  :)

Jason
Gerald Pfeifer - June 7, 2012, 1:29 p.m.
On Mon, 4 Jun 2012, Steven Bosscher wrote:
> How about this:

Looks good to me.  Do I understand correctly that this is a change
in GCC 4.7.1 and later (but not GCC 4.7.0)?  If that's the case, can
you note this, for example by prepending "[GCC 4.7.1 and later]" to
your snippet?

Gerald
Joseph S. Myers - June 12, 2012, 3:18 p.m.
On Mon, 4 Jun 2012, Jason Merrill wrote:

> On 06/04/2012 04:30 PM, Steven Bosscher wrote:
> > How about this:
> 
> OK, looks good.
> 
> > There is still a file config/arm/aout.h
> 
> I noticed that, but it seems to be used together with elfos.h, so it seems not
> to be a.out-specific.  :)

I commented in <http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01218.html> 
on how arm/aout.h could safely and usefully be merged into arm/arm.h (and 
on a few other oddities in the configuration of various ARM targets).

Patch

Index: htdocs/gcc-4.7/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v
retrieving revision 1.110
diff -u -r1.110 changes.html
--- htdocs/gcc-4.7/changes.html	1 Jun 2012 09:27:10 -0000	1.110
+++ htdocs/gcc-4.7/changes.html	4 Jun 2012 20:29:09 -0000
@@ -15,6 +15,14 @@ 
 <h2>Caveats</h2>

   <ul>
+    <li><p>The <code>-fconserve-space</code> flag has been
+    deprecated.  The flag had no effect for most targets: only
+    targets without a global <code>.bss</code> section and without
+    support for switchable sections.  Furthermore, the flag only
+    had an effect for G++, where it could result in wrong semantics
+    (please refer to the GCC manual for further details).
+    The flag will be removed in GCC 4.8</p></li>
+
     <li><p>Support for a number of older systems and recently
     unmaintained or untested target ports of GCC has been declared
     obsolete in GCC 4.7.  Unless there is activity to revive them, the