[PATCH] [Fix] Allow pointer types as access elements and compare them correct

Tobias Grosser tobias at grosser.es
Fri Sep 19 01:56:25 PDT 2014


On 19/09/2014 10:54, Johannes Doerfert wrote:
> On 09/19, Tobias Grosser wrote:
>> Very nice fix and test case.
>>
>> There is a minor nit in the test case and you may want to mention that you choose i8 as the max type,
> Updated.
>
>> ================
>> Comment at: test/Isl/CodeGen/aliasing_different_pointer_types.ll:1
>> @@ +1,2 @@
>> +; RUN: opt %loadPolly -polly-code-generator=isl -polly-codegen-isl -S < %s | FileCheck %s
>> +;
>> ----------------
>> -polly-code-generator=isl is not needed.
> Not true, it is. Otherwise the alias RTC's will be disabled.

Ah, that is surprising.

Anyhow, just leave it in. It will disappear when we remove CLooG.

Cheers,
Tobias




More information about the llvm-commits mailing list