<div dir="ltr">Hi Johannes,<br><br>I have rebased my GSoC project totally on top of these three patches and the polly trunk, it works well at the moment. Very nice patches!<br><br>As Tobias has explained, I compute an entirely new ast node and use the islcodegeneration to translate it into LLVM IR. That's why I always have the context from which I can get the isl_ast_build.<br>
<br>Thanks,<br>Yabin<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-08-02 1:27 GMT+02:00 Johannes Doerfert <span dir="ltr"><<a href="mailto:jdoerfert@codeaurora.org" target="_blank">jdoerfert@codeaurora.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Please see the attached patches<br>
<div class="im HOEnZb"><br>
--<br>
<br>
Johannes Doerfert<br>
Employee of Qualcomm Innovation Center, Inc.<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation<br>
<br>
<br>
-----Original Message-----<br>
From: Tobias Grosser [mailto:<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>]<br>
</div><div class="im HOEnZb">Sent: Thursday, July 31, 2014 3:48 PM<br>
To: Johannes Doerfert<br>
Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
Subject: Re: [Polly] Generating code for changed memory accesses<br>
<br>
</div><div class="HOEnZb"><div class="h5">On 01/08/2014 00:42, Johannes Doerfert wrote:<br>
>>> 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?<br>

> There are two problems, but if you just want to keep these tests you just need to address the first one:<br>
>    1) Change the context from { [] } to { : }<br>
>    2) As soon as scev codegen is turned on and we have new access functions we crash.<br>
<br>
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).<br>
<br>
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.<br>
<br>
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).<br>
<br>
Cheers,<br>
Tobias<br>
<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>