<div dir="ltr">On Tue, Sep 3, 2013 at 1:03 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The tablegen for the format attribute claims the first argument<br>
expected is a string.  However, the attribute actually takes an<br>
unresolved identifier instead of a string literal.  This patch<br>
clarifies that by making the attribute take an IdentifierArgument, and<br>
updates associated code.<br>
<br>
Since this is the only case where the parser was doing something<br>
special for StringArguments, I updated the tablegen emitter to remove<br>
complexity.<br><br></blockquote><div><br></div><div>Is there any reason to not just make FormatAttr::getType() return an IdentifierInfo?</div><div><br></div><div>Otherwise, looks fine.</div><div><br></div><div>-Eli</div>
</div></div></div>