diff mbox

Remove -fconserve-space

Message ID alpine.LNX.2.00.1208260214080.5727@ghan.fvgr
State New
Headers show

Commit Message

Gerald Pfeifer Aug. 26, 2012, 12:15 a.m. UTC
On Wed, 6 Jun 2012, Steven Bosscher wrote:
> The attached patch removes the -fconserve-space flag, as discussed last 
> week. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk?

How about the following for the release notes?

(Happy to use a different/better rationale.  This one's borrowed
from the old documentation. ;)

Gerald

Comments

Gerald Pfeifer Nov. 4, 2012, 2:13 p.m. UTC | #1
Hi there,

any comments on my proposed release notes entry below?

Gerald

On Sun, 26 Aug 2012, Gerald Pfeifer wrote:
>> The attached patch removes the -fconserve-space flag, as discussed last 
>> week. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk?
> 
> How about the following for the release notes?
> 
> (Happy to use a different/better rationale.  This one's borrowed
> from the old documentation. ;)
> 
> Gerald
> Index: changes.html
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
> retrieving revision 1.17
> diff -u -3 -p -r1.17 changes.html
> --- changes.html	20 Aug 2012 12:23:39 -0000	1.17
> +++ changes.html	26 Aug 2012 00:11:29 -0000
> @@ -38,7 +38,7 @@ explicit use of vector types may be inco
>  built with older versions of GCC.  Auto-vectorized code is not affected
>  by this change.</p>
>  
> -<h2>General Optimizer Improvements</h2>
> +<h2>General Optimizer Improvements (and Changes)</h2>
>  
>    <ul>
>      <li>A new option <code>-ftree-partial-pre</code> was added to control
> @@ -46,6 +46,9 @@ by this change.</p>
>        This option is enabled by default at the <code>-O3</code> optimization
>        level, and it makes PRE more aggressive.
>      </li>
> +    <li>The option <code>-fconserve-space</code> has been removed; it
> +    was no longer useful on most targets since GCC supports putting
> +    variables into BSS without making them common.</li>
>    </ul>
Jason Merrill Nov. 5, 2012, 2:01 a.m. UTC | #2
That looks good to me.

Jason
diff mbox

Patch

Index: changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
retrieving revision 1.17
diff -u -3 -p -r1.17 changes.html
--- changes.html	20 Aug 2012 12:23:39 -0000	1.17
+++ changes.html	26 Aug 2012 00:11:29 -0000
@@ -38,7 +38,7 @@  explicit use of vector types may be inco
 built with older versions of GCC.  Auto-vectorized code is not affected
 by this change.</p>
 
-<h2>General Optimizer Improvements</h2>
+<h2>General Optimizer Improvements (and Changes)</h2>
 
   <ul>
     <li>A new option <code>-ftree-partial-pre</code> was added to control
@@ -46,6 +46,9 @@  by this change.</p>
       This option is enabled by default at the <code>-O3</code> optimization
       level, and it makes PRE more aggressive.
     </li>
+    <li>The option <code>-fconserve-space</code> has been removed; it
+    was no longer useful on most targets since GCC supports putting
+    variables into BSS without making them common.</li>
   </ul>