[llvm] r272228 - [cpu-detection] Add missing break statements in outer switches

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 17:08:15 PDT 2016


Author: asbirlea
Date: Wed Jun  8 19:08:15 2016
New Revision: 272228

URL: http://llvm.org/viewvc/llvm-project?rev=272228&view=rev
Log:
[cpu-detection] Add missing break statements in outer switches

Summary:
Break on all switch cases for outer and inner switches.
No functionality changed.

Reviewers: llvm-commits, sanjoy

Differential Revision: http://reviews.llvm.org/D21158

Modified:
    llvm/trunk/lib/Support/Host.cpp

Modified: llvm/trunk/lib/Support/Host.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Host.cpp?rev=272228&r1=272227&r2=272228&view=diff
==============================================================================
--- llvm/trunk/lib/Support/Host.cpp (original)
+++ llvm/trunk/lib/Support/Host.cpp Wed Jun  8 19:08:15 2016
@@ -320,6 +320,7 @@ getIntelProcessorTypeAndSubtype(unsigned
       *Type = INTEL_i486;
       break;
     }
+    break;
   case 5:
     switch (Model) {
     case 1: // Pentium OverDrive processor for Pentium processor (60, 66),
@@ -341,6 +342,7 @@ getIntelProcessorTypeAndSubtype(unsigned
       *Type = INTEL_PENTIUM;
       break;
     }
+    break;
   case 6:
     switch (Model) {
     case 0x01: // Pentium Pro processor
@@ -537,6 +539,7 @@ getIntelProcessorTypeAndSubtype(unsigned
       *Type = INTEL_PENTIUM_PRO;
       break;
     }
+    break;
   case 15: {
     switch (Model) {
     case 0: // Pentium 4 processor, Intel Xeon processor. All processors are
@@ -572,6 +575,7 @@ getIntelProcessorTypeAndSubtype(unsigned
           ((Features & (1 << FEATURE_EM64T)) ? INTEL_X86_64 : INTEL_PENTIUM_IV);
       break;
     }
+    break;
   }
   default:
     break; /*"generic"*/
@@ -589,6 +593,7 @@ static void getAMDProcessorTypeAndSubtyp
   switch (Family) {
   case 4:
     *Type = AMD_i486;
+    break;
   case 5:
     *Type = AMDPENTIUM;
     switch (Model) {
@@ -606,9 +611,8 @@ static void getAMDProcessorTypeAndSubtyp
     case 10:
       *Subtype = AMDPENTIUM_GEODE;
       break; // "geode"
-    default:
-      break;
     }
+    break;
   case 6:
     *Type = AMDATHLON;
     switch (Model) {
@@ -623,9 +627,8 @@ static void getAMDProcessorTypeAndSubtyp
     case 10:
       *Subtype = AMDATHLON_XP;
       break; // "athlon-xp"
-    default:
-      break;
     }
+    break;
   case 15:
     *Type = AMDATHLON;
     if (Features & (1 << FEATURE_SSE3)) {
@@ -643,6 +646,7 @@ static void getAMDProcessorTypeAndSubtyp
       *Subtype = AMDATHLON_64;
       break; // "athlon64"
     }
+    break;
   case 16:
     *Type = AMDFAM10H; // "amdfam10"
     switch (Model) {
@@ -655,9 +659,8 @@ static void getAMDProcessorTypeAndSubtyp
     case 8:
       *Subtype = AMDFAM10H_ISTANBUL;
       break;
-    default:
-      break;
     }
+    break;
   case 20:
     *Type = AMDFAM14H;
     *Subtype = AMD_BTVER1;




More information about the llvm-commits mailing list