<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 17, 2014 at 11:32 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div><p dir="ltr"><span style="color:rgb(34,34,34)">LoopHintAttr should track an Expr*, in order to provide source fidelity and to support arbitrary constant expressions and template instantiation.  Keeping parens there should then not require any extra effort - but I think this is fairly low-value so I'm happy either way.</span></p>


</div></div></blockquote><div>Ah, I see.  Tyler Nowicki is working on a patch which adds support for expressions in loop hint pragmas.  Looks like his patch adds an Expr member to the loop hint attr.  When that lands, paren tracking can be done as you suggest which will be much cleaner.</div>
<div><br></div><div>Thanks,</div><div>Mark</div>
</div><br></div></div>