This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [GSoC] Addition of ISL AST generation to Graphite


On 16/07/2014 11:36, Richard Biener wrote:
On Tue, Jul 15, 2014 at 5:34 PM, Tobias Grosser <tobias@grosser.es> wrote:
On 15/07/2014 17:02, Roman Gareev wrote:

I'm seeing the error:

gcc/graphite-isl-ast-to-gimple.c:31:25: warning: isl/val_gmp.h: No such
file
or directory

when building for aarch64.

isl/val_gmp.h is included in 0.12 AFAICS so perhaps we should demand 0.12
instead of 0.11 ?


According to isl's ChangeLog, isl_val abstraction was added in version
0.12. Therefore, I think it would be right to demand on 0.12.

Tobias, what do you think about this? Is this fine for the backend,
which uses CLooG to generate Gimple code?


I think so. The latest release of CLooG (0.18.1) was released with isl
0.12.1 and is part of ftp://gcc.gnu.org/pub/gcc/infrastructure/. So
requiring isl 0.12.1 sounds reasonable.

Could you prepare such a patch?

Note that we also still accept CLooG 0.17.0.

In this case we need to update the minimal required CLooG version to CLooG 0.18.1, right?

> Btw, it's unfortunate that
ISL 0.13 cannot be used because it dropped some APIs we use
(it's important for testing on branches that a single cloog/isl version
can be used to bootstrap and test on active branches and trunk).

Right. I also believe this is very unfortunate. I copy Sven to let him now that dropping APIs too quickly regularly causes issues.

Cheers,
Tobias


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]