(patch,rfc) s/gimple/gimple */

Trevor Saunders tbsaunde@tbsaunde.org
Sun Sep 20 02:03:00 GMT 2015


On Fri, Sep 18, 2015 at 09:32:37AM -0600, Jeff Law wrote:
> On 09/18/2015 07:32 AM, Trevor Saunders wrote:
> >On Wed, Sep 16, 2015 at 03:11:14PM -0400, David Malcolm wrote:
> >>On Wed, 2015-09-16 at 09:16 -0400, Trevor Saunders wrote:
> >>>Hi,
> >>>
> >>>I gave changing from gimple to gimple * a shot last week.  It turned out
> >>>to be not too hard.  As you might expect the patch is huge so its
> >>>attached compressed.
> >>>
> >>>patch was bootstrapped + regtested on x86_64-linux-gnu, and run through
> >>>config-list.mk.  However I needed to update it some for changes made
> >>>while testing.  Do people want to make this change now?  If so I'll try
> >>>and commit the patch over the weekend when less is changing.
> >>
> >>
> >>FWIW there are some big changes in gcc/tree-vect-slp.c:vectorizable_load
> >>that looks like unrelated whitespace changes, e.g. the following (and
> >>there are some followup hunks).  Did something change underneath, or was
> >>there a stray whitespace cleanup here?  (I skimmed through the patch,
> >>and this was the only file I spotted where something looked wrong)
> >
> >yeah, it was a stray whitespace cleanup, but I reverted it.
> >
> >Given the few but only positive comments I've seen I'm planning to
> >commit this over the weekend.
> That works for me.

ok, its committed now :) I'm sure I broke something given the size so
please test :)  I tried to use git send-email first but of course the
patch was too big, so if you want to see it look at svn :)

Trev

> 
> jeff



More information about the Gcc-patches mailing list