Removed the -Wno-switch compiler option for the Clang builds on Linux

Fix all these warnings in the JIT that were disabled by the -Wno-switch option
Fix the two new warnings in Strike/strike.cpp

1 files changed, 17 insertions, 1 deletions

diff --git a/src/jit/assertionprop.cpp b/src/jit/assertionprop.cpp
index 409fc64542..fccec98089 100644
--- a/src/jit/assertionprop.cpp
+++ b/src/jit/assertionprop.cpp
@@ -1449,6 +1449,10 @@ void Compiler::optDebugCheckAssertions(unsigned index)
                }
            }
            break;
+
+        default:
+            // for all other 'assertion->op2.kind' values we don't check anything
+           break;
         }
     }
 }
@@ -1743,7 +1747,7 @@ void Compiler::optAssertionGen(GenTreePtr tree)
     // For most of the assertions that we create below 
     // the assertion is true after the tree is processed
     bool assertionProven = true;
-    unsigned assertionIndex  = 0;  
+    unsigned assertionIndex = NO_ASSERTION_INDEX; 
     switch (tree->gtOper)
     {
     case GT_ASG:
@@ -1814,6 +1818,10 @@ void Compiler::optAssertionGen(GenTreePtr tree)
     case GT_JTRUE:
         assertionIndex = optAssertionGenJtrue(tree);
         break;
+
+    default:
+        // All other gtOper node kinds, leave 'assertionIndex' = NO_ASSERTION_INDEX
+        break;
     }
 
     // For global assertion prop we must store the assertion number in the tree node
@@ -3771,6 +3779,10 @@ EXPSET_TP Compiler::optImpliedByConstAssertion(AssertionDsc* constAssertion)
             usable = ((impAssertion->assertionKind == OAK_EQUAL) && (impAssertion->op2.u1.iconVal == iconVal)) ||
                      ((impAssertion->assertionKind == OAK_NOT_EQUAL) && (impAssertion->op2.u1.iconVal != iconVal));
             break;
+
+        default:
+            // leave 'usable' = false;
+            break;
         }
 
         if (usable)
@@ -3932,6 +3944,10 @@ EXPSET_TP Compiler::optImpliedByCopyAssertion(AssertionDsc* copyAssertion, Asser
                           (depAssertLclNum == impAssertion->op1.lcl.lclNum && depAssertSsaNum == impAssertion->op1.lcl.ssaNum));
             }
             break;
+
+        default:
+            // leave 'usable' = false;
+            break;
         }
             
         if (usable)