[Patch]"Sema::checkFormatStringExpr" should check multiple "FormatAttr".

[Fuyao Zhao]

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.


On Tue, Sep 3, 2013 at 7:46 AM, Rafael Espíndola <rafael.espindola at gmail.com
> wrote:

> What is the expected meaning of
>
> const char* f2(const char *s1, const char *s2)
> __attribute__((format_arg(1)));
> const char* f2(const char *s1, const char *s2)
> __attribute__((format_arg(2)));
>
> Can you add a test showing we do the correct thing?
>
> On 2 September 2013 19:36, Fuyao Zhao <fuyaoz at google.com> wrote:
> > Hi,
> >
> > Functions could have multiple "FormatAttr" attributes, but
> > "Sema::checkFormatStringExpr" only checks last one. I attached the patch
> in
> > the mail.
> > --
> > Fuyao Zhao
> >
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
>



-- 
Fuyao
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130917/1c3907e3/attachment.html>