<div dir="ltr">Hi Tyler,<div><br></div><div>Please add checks for input numbers - e.g., adding #pragma loop interleave(3000000000) leads to metadata with negative number !{metadata !"llvm.vectorizer.unroll", i32 -1294967296}</div>
<div>CGStmt.cpp:585 - please add a space after //<br></div><div><br></div><div>Do you plan to support substitution of integer template parameters into pragma? Here is example:</div><div><div>>cat ploop.cpp</div><div>template <int VLEN> void while_test(int *List, int Length) {</div>
<div>  int i = 0;</div><div>  #pragma loop vectorize(VLEN)</div><div>  while(i < Length) {</div><div>    List[i] = i*2;</div><div>    i++;</div><div>  }</div><div>}</div><div>int main() {</div><div>    int L[100];</div>
<div>    while_test<4> (L, 100);</div><div>    return 0;</div><div>}</div></div><div><br></div><div>ParsePragma.cpp:1619: // Read '('  </div><div>This looks like a good place to use BalancedDelimiterTracker for parsing '(' and ')'.</div>
<div><br></div><div><span style="font-family:arial,sans-serif;font-size:12.800000190734863px">Regarding contradictory directives - I think they would be better to detect them/emit errors in front-end, so that  their uses in back-end can assume they are correct.</span><br>
</div><div><span style="font-family:arial,sans-serif;font-size:12.800000190734863px"><br></span></div><div><font face="arial, sans-serif">test/CodeGen/pragma-loop.cpp:69</font><br></div><div><font face="arial, sans-serif">// Verify c11 for range loop is recognized after<br>
</font></div><div><font face="arial, sans-serif">I guess this comment should say 'c++11'</font></div><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">Best regards,</font></div>
<div><font face="arial, sans-serif">Alexander</font></div><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif"><br></font></div><div class="gmail_extra"><br><br><div class="gmail_quote">
2014-04-28 5:35 GMT+04:00 Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class=""><br><div class="gmail_quote">On Sun, Apr 27, 2014 at 5:00 PM, Tyler Nowicki <span dir="ltr"><<a href="mailto:tnowicki@apple.com" target="_blank">tnowicki@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">- I don’t know Doug or Richard, could you add them to the cc-list.</blockquote></div><div class="gmail_extra"><br></div>
</div>
Adding Doug. ;]<br><br></div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>