Thanks for the fix. It sounds like GCC 4.8 didn’t like it either. Ben
On Mar 20, 2014, at 7:22 AM, Aaron Ballman <[email protected]> wrote: > Author: aaronballman > Date: Thu Mar 20 09:22:33 2014 > New Revision: 204345 > > URL: http://llvm.org/viewvc/llvm-project?rev=204345&view=rev > Log: > Silencing an MSVC warning about not all control paths returning a value. No > functional change intended. > > Modified: > cfe/trunk/lib/Lex/HeaderSearch.cpp > > Modified: cfe/trunk/lib/Lex/HeaderSearch.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/HeaderSearch.cpp?rev=204345&r1=204344&r2=204345&view=diff > ============================================================================== > --- cfe/trunk/lib/Lex/HeaderSearch.cpp (original) > +++ cfe/trunk/lib/Lex/HeaderSearch.cpp Thu Mar 20 09:22:33 2014 > @@ -1156,6 +1156,7 @@ bool HeaderSearch::loadModuleMapFile(con > case LMM_InvalidModuleMap: > return true; > } > + llvm_unreachable("Unknown load module map result"); > } > > HeaderSearch::LoadModuleMapResult > > > _______________________________________________ > 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
