Hi Tobi & Sebastian,<br><br>Sorry, I added static removal by mistake.<br><br>> Also could you prepare a patch to disable AA with -polly-ignore-aliasing<br><div id=":2x8">> Please test and fix any other places using AA with -polly-ignore-aliasing.<br>
<br>OK, I'll try.<br></div><br><div class="gmail_quote">2013/6/2 Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 06/02/2013 06:32 AM, Sebastian Pop wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Dmitry,<br>
<br>
The patch looks good to me.  Please allow Tobi to also comment on your patch<br>
before committing.<br>
<br>
Dmitry N. Mikushin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-static cl::opt<bool><br>
+cl::opt<bool><br>
</blockquote>
<br>
Why do you remove static here? Your change does not touch any other files.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  IgnoreAliasing("polly-ignore-<u></u>aliasing",<br>
                 cl::desc("Ignore possible aliasing of the array bases"),<br>
                 cl::Hidden, cl::init(false), cl::cat(PollyCategory));<br>
</blockquote>
<br>
<br>
Also could you prepare a patch to disable AA with -polly-ignore-aliasing<br>
<br>
   AA = &getAnalysisID<AliasAnalysis>(<u></u>AliasAnalysis::getClassPassID(<u></u>));<br>
<br>
Please test and fix any other places using AA with -polly-ignore-aliasing.<br>
<br>
Also please add [polly] on the subject line on patches for polly.<br>
</blockquote>
<br></div>
I agree with Sebastian comments. After addressing them, the patch seems good to go.<br>
<br>
Tobi<br>
<br>
</blockquote></div><br>