On 07/02/2015 09:09 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 2:03 PM, Ramana Radhakrishnan
<ramana....@googlemail.com> wrote:
How about a testcase or 2 or mentioning if it is covered by existing
testcases ?
The patch fixes a test in testsuite/gcc.dg/graphite/ when removing the
use of limit_scops().
Maybe the commit message could contain the name of the test that it fixed.
Right. I think just adding the missing information to the commit message
will be enough.
The patch that removes limit_scops() is in my opinion trivial, and
will be submitted for review once we fixed all the errors it can cause
(code gen, scop translation to polyhedral, etc.)
We will also fix bootstrap with graphite enabled, and then we will fix
all problems in bootstrap with limit_scops() removed.
I will also add a buildbot tracking nightly bootstraps with -floop-*
and -fgraphite-identity.
Sounds great. Nice to see more graphite activity again.
Best,
Tobias
