Evaluate expressions that delimit GCC switch/case ranges

Type casts may reasonably occur in here (and maybe also other expressions that
evaluate to constants). The regression test addionally covers the case of enum
values being used, which works fine (even without this change).

Author: tautschnig

regression/cbmc/gcc_switch_case_range2/main.c

@@ -0,0 +1,32 @@
+typedef enum
+{
+  VALUE_1,
+  VALUE_2,
+  VALUE_3,
+  VALUE_4
+} values;
+
+int main()
+{
+  unsigned x;
+  switch(x)
+  {
+  case VALUE_1:
+    __CPROVER_assert(0, "0 works");
+    break;
+#ifdef __GNUC__
+  case VALUE_2 ... 3:
+#else
+  case VALUE_2:
+  case VALUE_3:
+  case VALUE_4:
+#endif
+    __CPROVER_assert(0, "... works");
+    break;
+  default:
+    __CPROVER_assert(0, "default works");
+    break;
+  }
+
+  return 0;
+}

regression/cbmc/gcc_switch_case_range2/test.desc

@@ -0,0 +1,9 @@
+CORE
+main.c
+
+^EXIT=10$
+^SIGNAL=0$
+^VERIFICATION FAILED$
+^\*\* 3 of 3 failed
+--
+^warning: ignoring

src/goto-programs/goto_convert.cpp

@@ -406,8 +406,8 @@ void goto_convertt::convert_gcc_switch_case_range(
     throw 0;
   }
 
-  const auto lb = numeric_cast<mp_integer>(code.op0());
-  const auto ub = numeric_cast<mp_integer>(code.op1());
+  const auto lb = numeric_cast<mp_integer>(simplify_expr(code.op0(), ns));
+  const auto ub = numeric_cast<mp_integer>(simplify_expr(code.op1(), ns));
 
   if(!lb.has_value() || !ub.has_value())
   {