Commit f0bc3323 by Marek Polacek Committed by Marek Polacek

re PR c/7652 (-Wswitch-break : Warn if a switch case falls through)

	PR c/7652
gcc/c-family/
	* c-ada-spec.c (dump_generic_ada_node): Add return.
gcc/
	* cselib.c (cselib_expand_value_rtx_1): Add return.
	* gengtype.c (dbgprint_count_type_at): Likewise.
	* hsa-gen.c (gen_hsa_insn_for_internal_fn_call): Likewise.
	* reg-stack.c (get_true_reg): Restructure to avoid fallthrough warning.

From-SVN: r239297
parent bbc824cd
2016-08-09 Marek Polacek <polacek@redhat.com>
PR c/7652
* cselib.c (cselib_expand_value_rtx_1): Add return.
* gengtype.c (dbgprint_count_type_at): Likewise.
* hsa-gen.c (gen_hsa_insn_for_internal_fn_call): Likewise.
* reg-stack.c (get_true_reg): Restructure to avoid fallthrough warning.
2016-08-09 Martin Jambor <mjambor@suse.cz> 2016-08-09 Martin Jambor <mjambor@suse.cz>
PR ipa/71981 PR ipa/71981
......
2016-08-09 Marek Polacek <polacek@redhat.com>
PR c/7652
* c-ada-spec.c (dump_generic_ada_node): Add return.
2016-08-09 Jason Merrill <jason@redhat.com> 2016-08-09 Jason Merrill <jason@redhat.com>
* c-cppbuiltin.c (c_cpp_builtins): Update __cpp_constexpr for * c-cppbuiltin.c (c_cpp_builtins): Update __cpp_constexpr for
......
...@@ -1862,6 +1862,7 @@ dump_generic_ada_node (pretty_printer *buffer, tree node, tree type, int spc, ...@@ -1862,6 +1862,7 @@ dump_generic_ada_node (pretty_printer *buffer, tree node, tree type, int spc,
case TREE_BINFO: case TREE_BINFO:
dump_generic_ada_node dump_generic_ada_node
(buffer, BINFO_TYPE (node), type, spc, limited_access, name_only); (buffer, BINFO_TYPE (node), type, spc, limited_access, name_only);
return 0;
case TREE_VEC: case TREE_VEC:
pp_string (buffer, "--- unexpected node: TREE_VEC"); pp_string (buffer, "--- unexpected node: TREE_VEC");
......
...@@ -1618,6 +1618,7 @@ cselib_expand_value_rtx_1 (rtx orig, struct expand_value_data *evd, ...@@ -1618,6 +1618,7 @@ cselib_expand_value_rtx_1 (rtx orig, struct expand_value_data *evd,
else else
return orig; return orig;
} }
return orig;
} }
CASE_CONST_ANY: CASE_CONST_ANY:
......
...@@ -175,6 +175,7 @@ dbgprint_count_type_at (const char *fil, int lin, const char *msg, type_p t) ...@@ -175,6 +175,7 @@ dbgprint_count_type_at (const char *fil, int lin, const char *msg, type_p t)
{ {
case TYPE_UNDEFINED: case TYPE_UNDEFINED:
nb_undefined++; nb_undefined++;
break;
case TYPE_SCALAR: case TYPE_SCALAR:
nb_scalar++; nb_scalar++;
break; break;
......
...@@ -5039,6 +5039,7 @@ gen_hsa_insn_for_internal_fn_call (gcall *stmt, hsa_bb *hbb) ...@@ -5039,6 +5039,7 @@ gen_hsa_insn_for_internal_fn_call (gcall *stmt, hsa_bb *hbb)
case IFN_FMIN: case IFN_FMIN:
case IFN_FMAX: case IFN_FMAX:
gen_hsa_insns_for_call_of_internal_fn (stmt, hbb); gen_hsa_insns_for_call_of_internal_fn (stmt, hbb);
break;
default: default:
HSA_SORRY_ATV (gimple_location (stmt), HSA_SORRY_ATV (gimple_location (stmt),
......
...@@ -423,23 +423,25 @@ get_true_reg (rtx *pat) ...@@ -423,23 +423,25 @@ get_true_reg (rtx *pat)
GET_MODE (subreg)); GET_MODE (subreg));
return pat; return pat;
} }
pat = &XEXP (*pat, 0);
break;
} }
case FLOAT: case FLOAT:
case FIX: case FIX:
case FLOAT_EXTEND: case FLOAT_EXTEND:
pat = & XEXP (*pat, 0); pat = &XEXP (*pat, 0);
break; break;
case UNSPEC: case UNSPEC:
if (XINT (*pat, 1) == UNSPEC_TRUNC_NOOP if (XINT (*pat, 1) == UNSPEC_TRUNC_NOOP
|| XINT (*pat, 1) == UNSPEC_FILD_ATOMIC) || XINT (*pat, 1) == UNSPEC_FILD_ATOMIC)
pat = & XVECEXP (*pat, 0, 0); pat = &XVECEXP (*pat, 0, 0);
return pat; return pat;
case FLOAT_TRUNCATE: case FLOAT_TRUNCATE:
if (!flag_unsafe_math_optimizations) if (!flag_unsafe_math_optimizations)
return pat; return pat;
pat = & XEXP (*pat, 0); pat = &XEXP (*pat, 0);
break; break;
default: default:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment