This'll probably end up with a GCC warning about missing return after the switch & will probably require an llvm_unreachable be placed after the switch to silence GCC.
On Wed, Oct 15, 2014 at 10:46 AM, Kaelyn Takata <[email protected]> wrote: > Author: rikka > Date: Wed Oct 15 12:46:18 2014 > New Revision: 219818 > > URL: http://llvm.org/viewvc/llvm-project?rev=219818&view=rev > Log: > Drop unneccessary default case from switch introduced in r219809 > > This silences: > ../tools/clang/tools/libclang/CIndex.cpp:6451:3: warning: default label > in switch which covers all enumeration values [-Wcovered-switch-default] > > Modified: > cfe/trunk/tools/libclang/CIndex.cpp > > Modified: cfe/trunk/tools/libclang/CIndex.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/CIndex.cpp?rev=219818&r1=219817&r2=219818&view=diff > > ============================================================================== > --- cfe/trunk/tools/libclang/CIndex.cpp (original) > +++ cfe/trunk/tools/libclang/CIndex.cpp Wed Oct 15 12:46:18 2014 > @@ -6448,8 +6448,6 @@ enum CX_StorageClass clang_Cursor_getSto > return CX_SC_Auto; > case SC_Register: > return CX_SC_Register; > - default: > - return CX_SC_Invalid; > } > } > > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
