<div dir="ltr">hi, it's just re-definition of the functions. I could look at where to add the test case for this but it should not be in my this change.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 3, 2013 at 7:46 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">What is the expected meaning of<br>
<br>
const char* f2(const char *s1, const char *s2) __attribute__((format_arg(1)));<br>
const char* f2(const char *s1, const char *s2) __attribute__((format_arg(2)));<br>
<br>
Can you add a test showing we do the correct thing?<br>
<div><div class="h5"><br>
On 2 September 2013 19:36, Fuyao Zhao <<a href="mailto:fuyaoz@google.com">fuyaoz@google.com</a>> wrote:<br>
> Hi,<br>
><br>
> Functions could have multiple "FormatAttr" attributes, but<br>
> "Sema::checkFormatStringExpr" only checks last one. I attached the patch in<br>
> the mail.<br>
> --<br>
> Fuyao Zhao<br>
><br>
><br>
</div></div>> _______________________________________________<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><br clear="all"><div><br></div>-- <br>Fuyao
</div>