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 <[email protected]> wrote: > > > > On Fri, Feb 22, 2013 at 9:15 AM, Michael Han <[email protected]>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 > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
