On Mon, Jan 16, 2012 at 10:56 PM, David Blaikie <[email protected]> wrote:
> Modified: cfe/trunk/lib/Sema/SemaCodeComplete.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaCodeComplete.cpp?rev=148292&r1=148291&r2=148292&view=diff > > ============================================================================== > --- cfe/trunk/lib/Sema/SemaCodeComplete.cpp (original) > +++ cfe/trunk/lib/Sema/SemaCodeComplete.cpp Tue Jan 17 00:56:22 2012 > @@ -596,8 +596,7 @@ > default: > return STC_Arithmetic; > } > - return STC_Other; > - > + > case Type::Complex: > return STC_Arithmetic; > > @@ -1370,8 +1369,8 @@ > case Sema::PCC_ForInit: > return LangOpts.CPlusPlus || LangOpts.ObjC1 || LangOpts.C99; > } > - > - return false; > + > + llvm_unreachable("Invalid ParserCompletionContext!"); > } > > static PrintingPolicy getCompletionPrintingPolicy(const ASTContext > &Context, > @@ -2793,7 +2792,7 @@ > case Decl::ObjCCategory: return CXCursor_ObjCCategoryDecl; > case Decl::ObjCCategoryImpl: return CXCursor_ObjCCategoryImplDecl; > // FIXME > - return CXCursor_UnexposedDecl; > + // return CXCursor_UnexposedDecl; > Based on the lines before this I don't think this is the right fix... The FIXME was that we represented ObjCCategory and ObjCCategoryImpl as UnexposedDecl; now that we don't it should just be deleted rather than commented out?
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
