[PATCH] Emit diagnostic on illegal calling convention
richard at metafoo.co.uk
Tue May 7 12:47:31 PDT 2013
I believe your check for IsStatic in CheckFunctionCCAttr is incorrect for
operator new/operator delete (which are implicitly static). Also, walking
the DeclaratorChunks will not do the right thing for a function type which
is produced by a type specifier (eg, using a typedef for a function type),
and the code will crash for a declaration which has no DeclaratorChunks.
Can you delay this checking until after the FunctionDecl is built? (That
would also make the CXXABI checking interface nicer; you could pass in the
FunctionDecl and ask "is this valid?")
On Mon, Apr 29, 2013 at 9:03 AM, Alexander Zinenko <ftynse at gmail.com> wrote:
> Hello, John!
> Do you have time to review this?
> On Thursday, April 18, 2013, Alexander Zinenko wrote:
>> > I would just suppress all this checking whenever the declaration has a
>> scope qualifier and let redeclaration checking handle it.
>> Right, redeclaration checking suffice here, thanks.
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits