[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