[patch][graphite] Rewrite of "out of graphite"
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 <firstname.lastname@example.org> 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
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.
P.S.: I will look at it over the weekend. But it definitely seems like a
big step forward.
> 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.
More information about the Gcc-patches