[polly] r281052 - FlattenAlgo: Ensure we _really_ obtain a param space

[Michael Kruse via llvm-commits]

Thank you for finding a fix.

This is somewhat strange. Isn't isl_union_map_get_space defined to
return a parameter space? What is different on those machines where it
happened?

Michael


2016-09-09 18:11 GMT+02:00 Tobias Grosser via llvm-commits
<llvm-commits at lists.llvm.org>:
> Author: grosser
> Date: Fri Sep  9 11:11:26 2016
> New Revision: 281052
>
> URL: http://llvm.org/viewvc/llvm-project?rev=281052&view=rev
> Log:
> FlattenAlgo: Ensure we _really_ obtain a param space
>
> This resolves "isl_space.c:1775: not a parameter space" errors I have seen
> on two systems.
>
> Modified:
>     polly/trunk/lib/Transform/FlattenAlgo.cpp
>
> Modified: polly/trunk/lib/Transform/FlattenAlgo.cpp
> URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Transform/FlattenAlgo.cpp?rev=281052&r1=281051&r2=281052&view=diff
> ==============================================================================
> --- polly/trunk/lib/Transform/FlattenAlgo.cpp (original)
> +++ polly/trunk/lib/Transform/FlattenAlgo.cpp Fri Sep  9 11:11:26 2016
> @@ -227,7 +227,8 @@ IslPtr<isl_union_map> tryFlattenSequence
>    auto ScatterSet =
>        give(isl_set_from_union_set(isl_union_map_range(Schedule.copy())));
>
> -  auto ParamSpace = give(isl_union_map_get_space(Schedule.keep()));
> +  auto ParamSpace =
> +      give(isl_space_params(isl_union_map_get_space(Schedule.keep())));
>    auto Dims = isl_set_dim(ScatterSet.keep(), isl_dim_set);
>    assert(Dims >= 2);
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits