
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div><div>Hi,</div><div><br></div><div>Thanks to everyone for all their feedback. Please review the updated patch.</div><div><br></div><div>I modified the implementation to use attributed statements on the loop stmts to pass loop hints to the code generator. The benefit of using this method is that c++11 loop attributes won’t take too much more work to support, such as defining the syntax. Also attributed statements are part of the AST, along with their data.</div><div><br></div><div><div>The syntax remains unchanged as:</div><div><br></div><div>#pragma loop vectorize(enable/disable/value) interleave(enable/disable/value)</div></div><div><br></div><div>In response to Alexey Bataev:</div><div>1. Multiple-inheritance: Fixed using attributes to pass loop hints instead.</div><div>2. Support range-based loop: Done.</div><div>3. Loop Attributes: In theory this should work, but its untested. I couldn’t find any attributes that apply to loops or statements in general.</div><div>4. Serialization/deserialization: Should be supported with attributed stmts.</div><div>5. Expressions: Fixed. The value is now interpreted as a constant integer expression.</div><div>6. Contradictory directives: I don’t think this is a problem because there are only really two options vectorize and interleave. Contradictions could be easily checked if you think it would be a problem.</div><div>7. ast-dump and ast-print: Should be supported on attributed stmts.</div><div>8. Templates: Done and tests have been added.</div></div><div><br></div><div>In response to Hal Finkel:</div><div>- I made the changes you suggested. If I missed anything please let me know.</div><div>- I think the documentation should be added in an separate commit. I’ll get started on that next.</div><div>- I don’t know Doug or Richard, could you add them to the cc-list.</div><div><br></div><div>Looking forward to more feedback,</div><div><br></div><div>Tyler</div><div><br></div><div></div></body></html>