[Polly] Generating code for changed memory accesses

Tobias Grosser tobias at grosser.es
Thu Jul 31 15:48:08 PDT 2014


On 01/08/2014 00:42, Johannes Doerfert wrote:
>>> To your problem: Why would they fail? I remember you mentioned something about a different context being expected by isl. What is the minimal set of changes necessary to move the tests to the IslCodegeneration? Just changes to the context fields in the .json files?
> There are two problems, but if you just want to keep these tests you just need to address the first one:
>    1) Change the context from { [] } to { : }
>    2) As soon as scev codegen is turned on and we have new access functions we crash.

The second already exists in the current code, so it is not really a 
regression (You can use an explicitly disable scev code generation if 
this helps to keep your internal tests running).

The first one seems easy to do. Could you move the tests and
mention in the commit message that this is the only change that
has been applied to these files.

The remaining patch should then not need to change these files, no? If 
this is the case, you can mention in the commit messages that the change 
is tested by existing test cases (maybe point to them).

Cheers,
Tobias




More information about the llvm-commits mailing list