Patchwork [wwwdocs] PATCH for Re: [PATCH] Remove matrix-reorg

login
register
mail settings
Submitter Gerald Pfeifer
Date Sept. 4, 2012, 11:01 a.m.
Message ID <alpine.LNX.2.00.1209041259240.4718@trevally.site>
Download mbox | patch
Permalink /patch/181541/
State New
Headers show

Comments

Gerald Pfeifer - Sept. 4, 2012, 11:01 a.m.
On Mon, 3 Sep 2012, Richard Guenther wrote:
> I'd not mention the command-line flags.

I was thinking to point out what to not use any longer, in case.
Doesn't that make sense for the release notes?

> They were not working correctly and they did not work with LTO
> which made them useless apart from for single-TU programs.

How about the following?

Gerald
Richard Guenther - Sept. 4, 2012, 12:32 p.m.
On Tue, 4 Sep 2012, Gerald Pfeifer wrote:

> On Mon, 3 Sep 2012, Richard Guenther wrote:
> > I'd not mention the command-line flags.
> 
> I was thinking to point out what to not use any longer, in case.
> Doesn't that make sense for the release notes?
> 
> > They were not working correctly and they did not work with LTO
> > which made them useless apart from for single-TU programs.
> 
> How about the following?

Looks good to me.

Thanks,
Richard.

> Gerald
> 
> Index: changes.html
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
> retrieving revision 1.26
> diff -u -3 -p -r1.26 changes.html
> --- changes.html	2 Sep 2012 15:56:24 -0000	1.26
> +++ changes.html	4 Sep 2012 10:59:32 -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,11 @@ 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 struct reorg and matrix reorg optimizations (command-line
> +    options <code>-fipa-struct-reorg</code> and 
> +    <code>-fipa-matrix-reorg</code>) have been removed.  They did not
> +    work correctly nor with link-time optimization (LTO), hence were only
> +    applicable to programs consisting of a single translation unit.</li>
>    </ul>
>  
>  
> 
>

Patch

Index: changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
retrieving revision 1.26
diff -u -3 -p -r1.26 changes.html
--- changes.html	2 Sep 2012 15:56:24 -0000	1.26
+++ changes.html	4 Sep 2012 10:59:32 -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,11 @@  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 struct reorg and matrix reorg optimizations (command-line
+    options <code>-fipa-struct-reorg</code> and 
+    <code>-fipa-matrix-reorg</code>) have been removed.  They did not
+    work correctly nor with link-time optimization (LTO), hence were only
+    applicable to programs consisting of a single translation unit.</li>
   </ul>