[PATCH] [Polly] Runtime alias checks

Tobias Grosser tobias at grosser.es
Wed Sep 17 02:27:58 PDT 2014


On 17/09/2014 11:17, Johannes Doerfert wrote:
> On 09/17, Tobias Grosser wrote:
>> On 17/09/2014 09:34, Johannes Doerfert wrote:
>>> The necessary isl patches are in, lets move this along too, shall we?
>>
>> Fine with me. To do so, these steps are necessary:
>>
>> 1) Update isl to 2c19ecd444095d6f560349018f68993bc0e03691
>>
>> In a single commit with the run-time checks as otherwise we get warnings
>> about unlisted enum values.
> Done.
>
>> 2) Check that Polly tests pass with the latest isl version
> I will, but I got a little "problem". Since we still have the cloog
> backend as default I would have to add -polly-code-generator=isl to all
> the alias tests. Should I do that or do you see another solution e.g.,
> make isl default ;)?

I don't see a better solution.

And yes, having to keep two code paths here adds annoying overhead. We 
should get rid of it quickly.

>> 3) Disable the run-time alias checks in case the cloog backend is used,
>>     but enable it in case the isl backend is used.
> Done. (I had to move the CodeGenChoice but I did it the same way as we
> moved VectorizerChoice).
>
>> The last point should make sure that the CLooG tester keep working, while
>> the isl testers already test our code.

Cool.

Cheers,
Tobias



More information about the llvm-commits mailing list