r215897 - Add the break in the switch case (even if there is llvm_unreachable. It will silent coverity warning CID 1231654

Sylvestre Ledru sylvestre at debian.org
Mon Aug 18 08:13:45 PDT 2014


Author: sylvestre
Date: Mon Aug 18 10:13:44 2014
New Revision: 215897

URL: http://llvm.org/viewvc/llvm-project?rev=215897&view=rev
Log:
Add the break in the switch case (even if there is llvm_unreachable. It will silent coverity warning CID 1231654

Modified:
    cfe/trunk/lib/Frontend/InitHeaderSearch.cpp

Modified: cfe/trunk/lib/Frontend/InitHeaderSearch.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitHeaderSearch.cpp?rev=215897&r1=215896&r2=215897&view=diff
==============================================================================
--- cfe/trunk/lib/Frontend/InitHeaderSearch.cpp (original)
+++ cfe/trunk/lib/Frontend/InitHeaderSearch.cpp Mon Aug 18 10:13:44 2014
@@ -390,7 +390,7 @@ AddDefaultCPlusPlusIncludePaths(const ll
   switch (os) {
   case llvm::Triple::Linux:
     llvm_unreachable("Include management is handled in the driver.");
-
+    break;
   case llvm::Triple::Win32:
     switch (triple.getEnvironment()) {
     default: llvm_unreachable("Include management is handled in the driver.");





More information about the cfe-commits mailing list