r204345 - Silencing an MSVC warning about not all control paths returning a value. No functional change intended.

[Ben Langmuir]

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 <aaron at aaronballman.com> 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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits