diff mbox

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

Message ID alpine.LNX.2.00.1209022039330.4718@trevally.site
State New
Headers show

Commit Message

Gerald Pfeifer Sept. 2, 2012, 6:42 p.m. UTC
Hi Richi,

On Fri, 10 Aug 2012, Richard Guenther wrote:
> This removes matrix-reorg which is today useless and possibly
> dangerous.  It follows struct-reorg down the kitchen-sink.

how about the following patch for the GCC 4.8 release notes?

Would you like to propose a (politically correct ;-) snippet
on why you removed the two ?

Gerald

Comments

Richard Biener Sept. 3, 2012, 9 a.m. UTC | #1
On Sun, 2 Sep 2012, Gerald Pfeifer wrote:

> Hi Richi,
> 
> On Fri, 10 Aug 2012, Richard Guenther wrote:
> > This removes matrix-reorg which is today useless and possibly
> > dangerous.  It follows struct-reorg down the kitchen-sink.
> 
> how about the following patch for the GCC 4.8 release notes?

I'd not mention the command-line flags.

> Would you like to propose a (politically correct ;-) snippet
> on why you removed the two ?

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

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	2 Sep 2012 18:39:27 -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 struct reorg and matrix reorg optimizations (command-line
> +    options <code>-fipa-struct-reorg</code> and 
> +    <code>-fipa-matrix-reorg</code>) have been removed.</li>
>    </ul>
>  
>  
> 
>
diff mbox

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	2 Sep 2012 18:39:27 -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 struct reorg and matrix reorg optimizations (command-line
+    options <code>-fipa-struct-reorg</code> and 
+    <code>-fipa-matrix-reorg</code>) have been removed.</li>
   </ul>