llvm dev - Jul 2007 - [LLVMdev] [PATCH] fix a "jump to case label crosses initialization of llvm::MVT::ValueType VT" error

Index: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
==================================================================---
llvm.orig/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp	2007-07-14
16:59:23.000000000 +0200
+++ llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp	2007-07-14 
16:59:52.000000000 +0200
@@ -696,7 +696,7 @@
     }
     }
     break;
-  case ISD::EH_RETURN:
+  case ISD::EH_RETURN: {
     MVT::ValueType VT = Node->getValueType(0);
     // The only "good" option for this node is to custom lower it.
     switch (TLI.getOperationAction(Node->getOpcode(), VT)) {
@@ -710,6 +710,7 @@
       Result = LegalizeOp(Node->getOperand(0));
       break;
     }
+    }
     break;
   case ISD::AssertSext:
   case ISD::AssertZext:

Thanks Holger, I've applied your patch to SVN.

Nick Lewycky

Holger Schurig wrote:
> Index: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
> ==================================================================> ---
llvm.orig/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp	2007-07-14
> 16:59:23.000000000 +0200
> +++ llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp	2007-07-14 
> 16:59:52.000000000 +0200
> @@ -696,7 +696,7 @@
>      }
>      }
>      break;
> -  case ISD::EH_RETURN:
> +  case ISD::EH_RETURN: {
>      MVT::ValueType VT = Node->getValueType(0);
>      // The only "good" option for this node is to custom lower
it.
>      switch (TLI.getOperationAction(Node->getOpcode(), VT)) {
> @@ -710,6 +710,7 @@
>        Result = LegalizeOp(Node->getOperand(0));
>        break;
>      }
> +    }
>      break;
>    case ISD::AssertSext:
>    case ISD::AssertZext: