[cfe-dev] [PATCH] extend __interface support
João Matos
ripzonetriton at gmail.com
Mon Aug 13 07:02:12 PDT 2012
On Mon, Aug 13, 2012 at 2:30 PM, David Robins <llvm at davidrobins.net> wrote:
> MSVC 10 (2010) emits (with /Wall; nothing without):
>
> int.cpp(4) : warning C4099: 'Y' : type name first seen using 'struct'
> now seen using 'class'
> int.cpp(3) : see declaration of 'Y'
>
No wonder I wasn't getting warnings, forgot to enable "-Wall" in my tests.
I can see some advantages in modelling this as a new tag type specifier.
Just from a quick glance at the code, there are some places in the parser
where we're assigning the expected name type specifier (struct / class /
enum) and with an attribute these cases might be ignored. With a new TST
value, this will trigger compiler warnings (and hopefully it will get fixed
:).
--
João Matos
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20120813/7d246ddb/attachment.html>
More information about the cfe-dev
mailing list