r175900 - [Sema] Semantic analysis for empty-declaration and attribute-declaration.

[Michael Han]

Hi David,

Sorry for breaking the build. Yes I think it is the appropriate fix. Thanks!

Michael.

On Fri, Feb 22, 2013 at 9:46 AM, David Blaikie <dblaikie at gmail.com> wrote:

>
>
>
> On Fri, Feb 22, 2013 at 9:15 AM, Michael Han <fragmentshaders at gmail.com>wrote:
>
>> Author: hanm
>> Date: Fri Feb 22 11:15:32 2013
>> New Revision: 175900
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=175900&view=rev
>> Log:
>> [Sema] Semantic analysis for empty-declaration and attribute-declaration.
>>
>> Introduce a new AST Decl node "EmptyDecl" to model empty-declaration.
>> Have attributes from attribute-declaration appertain
>> to the EmptyDecl node by creating the AST representations of these
>> attributes and attach them to the EmptyDecl node so these
>> attributes can be sema checked just as attributes attached to "normal"
>> declarations.
>>
>>
>>
> This breaks the -Werror build of libclang:
>
> llvm/src/tools/clang/tools/libclang/CIndex.cpp:4428:11: error: enumeration
> value 'Empty' not handled in switch [-Werror,-Wswitch]
>   switch (D->getKind()) {
>           ^
>
> I've 'fixed' (more like suppressed) this in r175902 - could you take a
> look & correct this as appropriate? (maybe add a libclang test? Not sure
> how that code is tested)
>
> - David
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130222/16ad415c/attachment.html>