<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 22, 2014 at 4:54 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"><div>Please review this updated patch. It includes the changes we discussed. Thanks for all your input!</div><span class="HOEnZb"><font color="#888888"><div>
</div></font></span></blockquote></div><div class="gmail_extra"><br></div>FWIW, I'm roughly happy with the syntax / naming direction this is heading in.</div><div class="gmail_extra"><br></div><div class="gmail_extra">
You probably want someone more familiar with the Clang internals (AST, pragma parsing) to review the patch in detail (Doug or Richard) but I'm somewhat concerned about the multiple inheritance. I don't think that's really the right way to put this into the AST, but it's been a while since I was lats AST hacking. =]</div>
<div class="gmail_extra"><br></div></div>