addressing comments

Author: joaosaffran

clang/lib/CodeGen/CGStmt.cpp

@@ -2276,27 +2276,18 @@ void CodeGenFunction::EmitSwitchStmt(const SwitchStmt &S) {
   // failure.
   llvm::BasicBlock *DefaultBlock = createBasicBlock("sw.default");
   SwitchInsn = Builder.CreateSwitch(CondV, DefaultBlock);
-  switch (HLSLControlFlowAttr) {
-  case HLSLControlFlowHintAttr::Microsoft_branch:
-  case HLSLControlFlowHintAttr::Microsoft_flatten: {
+  if(HLSLControlFlowAttr != HLSLControlFlowHintAttr::SpellingNotCalculated) {
     llvm::MDBuilder MDHelper(CGM.getLLVMContext());
-
     llvm::ConstantInt *BranchHintConstant =
         HLSLControlFlowAttr ==
                 HLSLControlFlowHintAttr::Spelling::Microsoft_branch
             ? llvm::ConstantInt::get(CGM.Int32Ty, 1)
             : llvm::ConstantInt::get(CGM.Int32Ty, 2);
-
-    SmallVector<llvm::Metadata *, 2> Vals(
+    llvm::Metadata * Vals[] =
         {MDHelper.createString("hlsl.controlflow.hint"),
-         MDHelper.createConstant(BranchHintConstant)});
+         MDHelper.createConstant(BranchHintConstant)};
     SwitchInsn->setMetadata("hlsl.controlflow.hint",
                             llvm::MDNode::get(CGM.getLLVMContext(), Vals));
-    break;
-  }
-  // This is required to avoid warnings during compilation
-  case HLSLControlFlowHintAttr::SpellingNotCalculated:
-    break;
   }
 
   if (PGO.haveRegionCounts()) {