On 16/07/2014 11:36, Richard Biener wrote:
On Tue, Jul 15, 2014 at 5:34 PM, Tobias Grosser <tob...@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