On Tue, Jan 8, 2013 at 1:18 PM, David Greene <[email protected]> wrote: > Return a value to avoid an empty return warning. > --- > llvm/tools/clang/lib/AST/ASTContext.cpp | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/llvm/tools/clang/lib/AST/ASTContext.cpp > b/llvm/tools/clang/lib/AST/ASTContext.cpp > index dc398f3..e0e2478 100644 > --- a/llvm/tools/clang/lib/AST/ASTContext.cpp > +++ b/llvm/tools/clang/lib/AST/ASTContext.cpp > @@ -4870,6 +4870,7 @@ static char getObjCEncodingForPrimitiveKind(const > ASTContext *C, > #include "clang/AST/BuiltinTypes.def" > llvm_unreachable("invalid builtin type for @encode"); > } > + return 0;
It looks like falling off the end of this switch should never happen, so there should be an llvm_unreachable rather than a return here. -Matt _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
