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

[Richard Guenther <rguenther at suse dot de>]

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>
>  
>  
> 
> 

-- 
Richard Guenther <rguenther@suse.de>
SUSE / SUSE Labs
SUSE LINUX Products GmbH - Nuernberg - AG Nuernberg - HRB 16746
GF: Jeff Hawn, Jennifer Guild, Felix Imend