[patch][graphite] Rewrite of "out of graphite"

Tobias Grosser grosser@fim.uni-passau.de
Wed Dec 3 19:14:00 GMT 2008


On Wed, 2008-12-03 at 13:57 -0500, Jack Howarth wrote:
> On Wed, Dec 03, 2008 at 12:44:27PM -0600, Sebastian Pop wrote:
> > Hi,
> > 
> > On Wed, Dec 3, 2008 at 4:22 AM, Richard Guenther <rguenther@suse.de> wrote:
> > > The middle-end parts could have been splitted ;)  The middle-end parts
> > > are ok, the graphite parts as well, but you may want to wait for more
> > > comments from other graphite reviewers.
> > >
> > 
> > Thanks for the review Richi.  Other Graphite reviewers, if you have
> > time, please send me comments on the patches.  I plan to commit
> > the previous patch to trunk on Friday or Monday to let you enough
> > time to review.
> > 
> > Here is another patch fixing id-1.f90 that is a type conversion
> > problem arising when expanding to RTL.  I committed this to the
> > graphite branch.  Okay for trunk after regtest passes on amd64-linux?
> > 
> > Thanks,
> > Sebastian Pop

Hey Sebastian,

I just tried it. There is one bug in build_scops_1 (I will work on it)
left, but is seems otherwise we build polyhedron!

That looks like really great work! Thanks a lot.

Tobi

P.S.: I will look at it over the weekend. But it definitely seems like a
big step forward.


> Sebastian,
>     Does graphite branch now build all of the polyhedron 2005 benchmarks
> at -O2 fgraphite-identity without compilation or execution problems? If so
> that's great.
>                     Jack



More information about the Gcc-patches mailing list