From bbfac6da602bfe02c347edcb1639aa62ce63c2d4 Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Tue, 27 Sep 2016 20:14:10 +0200 Subject: [PATCH] auto-inc-dec.c (try_merge): Remove break after return. * auto-inc-dec.c (try_merge): Remove break after return. * cselib.c (autoinc_split): Likewise. * explow.c (promote_mode): Likewise. * fixed-value.c (fixed_arithmetic): Likewise. * hsa.c (hsa_internal_fn::get_arity): Likewise. * rtlanal.c (modified_between_p, modified_in_p): Likewise. * trans-mem.c (get_attrs_for): Likewise. * tree-if-conv.c (if_convertible_stmt_p): Likewise. * tree-vrp.c (simplify_stmt_using_ranges): Likewise. * config/aarch64/aarch64-builtins.c (aarch64_fold_builtin): Likewise. * config/aarch64/aarch64.c (aarch64_get_condition_code_1): Likewise. * config/c6x/c6x.c (c6x_get_unit_specifier): Likewise. * config/cr16/cr16.c (legitimate_pic_operand_p): Likewise. * config/cris/cris.c (cris_op_str): Likewise. * config/mn10300/mn10300.c (cc_flags_for_code): Likewise. * config/tilepro/tilepro.c (tilepro_emit_setcc_internal_di): Likewise. c-family/ * c-ada-spec.c (print_ada_declaration): Remove break after return. objc/ * objc-act.c (continue_class): Remove break after return. (objc_maybe_printable_name): Likewise. fortran/ * dependency.c (gfc_dep_compare_expr): Remove break after return. * frontend-passes.c (optimize_op): Likewise. * interface.c (gfc_current_interface_head): Likewise. * symbol.c (check_conflict): Likewise. * trans-intrinsic.c (build_fix_expr): Likewise. ada/ * terminals.c (is_gui_app): Remove break after return. From-SVN: r240554 --- gcc/ChangeLog | 19 +++++++++++++++++++ gcc/ada/ChangeLog | 4 ++++ gcc/ada/terminals.c | 7 ------- gcc/auto-inc-dec.c | 8 -------- gcc/c-family/ChangeLog | 4 ++++ gcc/c-family/c-ada-spec.c | 1 - gcc/config/aarch64/aarch64-builtins.c | 1 - gcc/config/aarch64/aarch64.c | 1 - gcc/config/c6x/c6x.c | 4 ---- gcc/config/cr16/cr16.c | 3 --- gcc/config/cris/cris.c | 12 ------------ gcc/config/mn10300/mn10300.c | 1 - gcc/config/tilepro/tilepro.c | 2 -- gcc/cselib.c | 1 - gcc/explow.c | 2 -- gcc/fixed-value.c | 7 ------- gcc/fortran/ChangeLog | 6 ++++++ gcc/fortran/dependency.c | 1 - gcc/fortran/frontend-passes.c | 1 - gcc/fortran/interface.c | 3 --- gcc/fortran/symbol.c | 1 - gcc/fortran/trans-intrinsic.c | 4 ---- gcc/hsa.c | 1 - gcc/objc/ChangeLog | 5 +++++ gcc/objc/objc-act.c | 6 ------ gcc/rtlanal.c | 2 -- gcc/trans-mem.c | 1 - gcc/tree-if-conv.c | 1 - gcc/tree-vrp.c | 1 - 29 files changed, 38 insertions(+), 72 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 39e18fdf0ec..47029b04339 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,22 @@ +2016-09-27 Jakub Jelinek + + * auto-inc-dec.c (try_merge): Remove break after return. + * cselib.c (autoinc_split): Likewise. + * explow.c (promote_mode): Likewise. + * fixed-value.c (fixed_arithmetic): Likewise. + * hsa.c (hsa_internal_fn::get_arity): Likewise. + * rtlanal.c (modified_between_p, modified_in_p): Likewise. + * trans-mem.c (get_attrs_for): Likewise. + * tree-if-conv.c (if_convertible_stmt_p): Likewise. + * tree-vrp.c (simplify_stmt_using_ranges): Likewise. + * config/aarch64/aarch64-builtins.c (aarch64_fold_builtin): Likewise. + * config/aarch64/aarch64.c (aarch64_get_condition_code_1): Likewise. + * config/c6x/c6x.c (c6x_get_unit_specifier): Likewise. + * config/cr16/cr16.c (legitimate_pic_operand_p): Likewise. + * config/cris/cris.c (cris_op_str): Likewise. + * config/mn10300/mn10300.c (cc_flags_for_code): Likewise. + * config/tilepro/tilepro.c (tilepro_emit_setcc_internal_di): Likewise. + 2016-09-27 Nathan Sidwell * internal-fn.h (IFN_UNIQUE_CODES, IFN_GOACC_LOOP_CODES, diff --git a/gcc/ada/ChangeLog b/gcc/ada/ChangeLog index 032d5cd7e9d..4f941ec8501 100644 --- a/gcc/ada/ChangeLog +++ b/gcc/ada/ChangeLog @@ -1,3 +1,7 @@ +2016-09-27 Jakub Jelinek + + * terminals.c (is_gui_app): Remove break after return. + 2016-09-26 Rainer Orth * gcc-interface/decl.c: Fix fall through comment formatting. diff --git a/gcc/ada/terminals.c b/gcc/ada/terminals.c index d72eb098850..31f21f7a6fd 100644 --- a/gcc/ada/terminals.c +++ b/gcc/ada/terminals.c @@ -289,34 +289,27 @@ is_gui_app (char *exe) { case IMAGE_SUBSYSTEM_UNKNOWN: return 1; - break; case IMAGE_SUBSYSTEM_NATIVE: return 1; - break; case IMAGE_SUBSYSTEM_WINDOWS_GUI: return 1; - break; case IMAGE_SUBSYSTEM_WINDOWS_CUI: return 0; - break; case IMAGE_SUBSYSTEM_OS2_CUI: return 0; - break; case IMAGE_SUBSYSTEM_POSIX_CUI: return 0; - break; default: /* Unknown, return GUI app to be preservative: if yes, it will be correctly launched, if no, it will be launched, and a console will be also displayed, which is not a big deal */ return 1; - break; } } diff --git a/gcc/auto-inc-dec.c b/gcc/auto-inc-dec.c index bf4959baabc..eb2dc21f801 100644 --- a/gcc/auto-inc-dec.c +++ b/gcc/auto-inc-dec.c @@ -658,25 +658,21 @@ try_merge (void) if (dump_file) fprintf (dump_file, "trying SIMPLE_PRE_INC\n"); return attempt_change (gen_rtx_PRE_INC (reg_mode, inc_reg), inc_reg); - break; case SIMPLE_POST_INC: /* size++ */ if (dump_file) fprintf (dump_file, "trying SIMPLE_POST_INC\n"); return attempt_change (gen_rtx_POST_INC (reg_mode, inc_reg), inc_reg); - break; case SIMPLE_PRE_DEC: /* --size */ if (dump_file) fprintf (dump_file, "trying SIMPLE_PRE_DEC\n"); return attempt_change (gen_rtx_PRE_DEC (reg_mode, inc_reg), inc_reg); - break; case SIMPLE_POST_DEC: /* size-- */ if (dump_file) fprintf (dump_file, "trying SIMPLE_POST_DEC\n"); return attempt_change (gen_rtx_POST_DEC (reg_mode, inc_reg), inc_reg); - break; case DISP_PRE: /* ++con */ if (dump_file) @@ -687,7 +683,6 @@ try_merge (void) inc_reg, inc_insn.reg1)), inc_reg); - break; case DISP_POST: /* con++ */ if (dump_file) @@ -698,7 +693,6 @@ try_merge (void) inc_reg, inc_insn.reg1)), inc_reg); - break; case REG_PRE: /* ++reg */ if (dump_file) @@ -709,7 +703,6 @@ try_merge (void) inc_reg, inc_insn.reg1)), inc_reg); - break; case REG_POST: /* reg++ */ if (dump_file) @@ -720,7 +713,6 @@ try_merge (void) inc_reg, inc_insn.reg1)), inc_reg); - break; } } diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog index e0b1333819e..f60f63eee25 100644 --- a/gcc/c-family/ChangeLog +++ b/gcc/c-family/ChangeLog @@ -1,3 +1,7 @@ +2016-09-27 Jakub Jelinek + + * c-ada-spec.c (print_ada_declaration): Remove break after return. + 2016-09-26 Thomas Preud'homme * c-common.c: Include memmodel.h. diff --git a/gcc/c-family/c-ada-spec.c b/gcc/c-family/c-ada-spec.c index 6a8e04b8d36..a5395b69489 100644 --- a/gcc/c-family/c-ada-spec.c +++ b/gcc/c-family/c-ada-spec.c @@ -2902,7 +2902,6 @@ print_ada_declaration (pretty_printer *buffer, tree t, tree type, int spc) pp_string (buffer, "-- skipped function type "); dump_generic_ada_node (buffer, t, type, spc, false, true); return 1; - break; case ENUMERAL_TYPE: if ((orig && TYPE_NAME (orig) && orig != TREE_TYPE (t)) diff --git a/gcc/config/aarch64/aarch64-builtins.c b/gcc/config/aarch64/aarch64-builtins.c index 822d608a5b3..b3625801761 100644 --- a/gcc/config/aarch64/aarch64-builtins.c +++ b/gcc/config/aarch64/aarch64-builtins.c @@ -1460,7 +1460,6 @@ aarch64_fold_builtin (tree fndecl, int n_args ATTRIBUTE_UNUSED, tree *args, { BUILTIN_VDQF (UNOP, abs, 2) return fold_build1 (ABS_EXPR, type, args[0]); - break; VAR1 (UNOP, floatv2si, 2, v2sf) VAR1 (UNOP, floatv4si, 2, v4sf) VAR1 (UNOP, floatv2di, 2, v2df) diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index c65b826b0cc..df6514d85e6 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -4433,7 +4433,6 @@ aarch64_get_condition_code_1 (enum machine_mode mode, enum rtx_code comp_code) default: return -1; - break; } return -1; diff --git a/gcc/config/c6x/c6x.c b/gcc/config/c6x/c6x.c index df6cb78ce62..305f31ea0b3 100644 --- a/gcc/config/c6x/c6x.c +++ b/gcc/config/c6x/c6x.c @@ -1987,17 +1987,13 @@ c6x_get_unit_specifier (rtx_insn *insn) case UNITS_DLS: case UNITS_D_ADDR: return 'd'; - break; case UNITS_L: case UNITS_LS: return 'l'; - break; case UNITS_S: return 's'; - break; case UNITS_M: return 'm'; - break; default: gcc_unreachable (); } diff --git a/gcc/config/cr16/cr16.c b/gcc/config/cr16/cr16.c index 2f75ca2301b..13e8e475ae9 100644 --- a/gcc/config/cr16/cr16.c +++ b/gcc/config/cr16/cr16.c @@ -1120,10 +1120,8 @@ legitimate_pic_operand_p (rtx x) { case SYMBOL_REF: return 0; - break; case LABEL_REF: return 0; - break; case CONST: /* REVISIT: Use something like symbol_referenced_p. */ if (GET_CODE (XEXP (x, 0)) == PLUS @@ -1134,7 +1132,6 @@ legitimate_pic_operand_p (rtx x) break; case MEM: return legitimate_pic_operand_p (XEXP (x, 0)); - break; default: break; } diff --git a/gcc/config/cris/cris.c b/gcc/config/cris/cris.c index c909453006d..7532b8e501a 100644 --- a/gcc/config/cris/cris.c +++ b/gcc/config/cris/cris.c @@ -544,11 +544,9 @@ cris_op_str (rtx x) { case PLUS: return "add"; - break; case MINUS: return "sub"; - break; case MULT: /* This function is for retrieving a part of an instruction name for @@ -560,46 +558,36 @@ cris_op_str (rtx x) case DIV: return "div"; - break; case AND: return "and"; - break; case IOR: return "or"; - break; case XOR: return "xor"; - break; case NOT: return "not"; - break; case ASHIFT: return "lsl"; - break; case LSHIFTRT: return "lsr"; - break; case ASHIFTRT: return "asr"; - break; case UMIN: /* Used to control the sign/zero-extend character for the 'E' modifier. BOUND has none. */ cris_output_insn_is_bound = 1; return "bound"; - break; default: return "Unknown operator"; - break; } } diff --git a/gcc/config/mn10300/mn10300.c b/gcc/config/mn10300/mn10300.c index e384796873a..81c8506a0ff 100644 --- a/gcc/config/mn10300/mn10300.c +++ b/gcc/config/mn10300/mn10300.c @@ -2693,7 +2693,6 @@ cc_flags_for_code (enum rtx_code code) case LT: /* N */ case GE: /* ~N */ return CC_FLAG_N; - break; case GT: /* ~(Z|(N^V)) */ case LE: /* Z|(N^V) */ diff --git a/gcc/config/tilepro/tilepro.c b/gcc/config/tilepro/tilepro.c index 33e49936536..b39eff4b6a8 100644 --- a/gcc/config/tilepro/tilepro.c +++ b/gcc/config/tilepro/tilepro.c @@ -2137,13 +2137,11 @@ tilepro_emit_setcc_internal_di (rtx res, enum rtx_code code, rtx op0, rtx op1) emit_insn (gen_insn_seq (tmp1, hi_half[0], hi_half[1])); emit_insn (gen_andsi3 (res, tmp0, tmp1)); return true; - break; case NE: emit_insn (gen_insn_sne (tmp0, lo_half[0], lo_half[1])); emit_insn (gen_insn_sne (tmp1, hi_half[0], hi_half[1])); emit_insn (gen_iorsi3 (res, tmp0, tmp1)); return true; - break; case LE: emit_insn (gen_insn_slte (tmp0, hi_half[0], hi_half[1])); emit_insn (gen_insn_seq (tmp1, hi_half[0], hi_half[1])); diff --git a/gcc/cselib.c b/gcc/cselib.c index 1277842783b..24e15bca211 100644 --- a/gcc/cselib.c +++ b/gcc/cselib.c @@ -805,7 +805,6 @@ autoinc_split (rtx x, rtx *off, machine_mode memmode) *off = GEN_INT (-GET_MODE_SIZE (memmode)); return XEXP (x, 0); - break; case PRE_INC: if (memmode == VOIDmode) diff --git a/gcc/explow.c b/gcc/explow.c index e935c307bcb..8dcef8bb350 100644 --- a/gcc/explow.c +++ b/gcc/explow.c @@ -802,7 +802,6 @@ promote_mode (const_tree type ATTRIBUTE_UNUSED, machine_mode mode, PROMOTE_MODE (mode, unsignedp, type); *punsignedp = unsignedp; return mode; - break; #ifdef POINTERS_EXTEND_UNSIGNED case REFERENCE_TYPE: @@ -810,7 +809,6 @@ promote_mode (const_tree type ATTRIBUTE_UNUSED, machine_mode mode, *punsignedp = POINTERS_EXTEND_UNSIGNED; return targetm.addr_space.address_mode (TYPE_ADDR_SPACE (TREE_TYPE (type))); - break; #endif default: diff --git a/gcc/fixed-value.c b/gcc/fixed-value.c index ad92c497d9f..ddb8ee112d5 100644 --- a/gcc/fixed-value.c +++ b/gcc/fixed-value.c @@ -730,35 +730,28 @@ fixed_arithmetic (FIXED_VALUE_TYPE *f, int icode, const FIXED_VALUE_TYPE *op0, { case NEGATE_EXPR: return do_fixed_neg (f, op0, sat_p); - break; case PLUS_EXPR: gcc_assert (op0->mode == op1->mode); return do_fixed_add (f, op0, op1, false, sat_p); - break; case MINUS_EXPR: gcc_assert (op0->mode == op1->mode); return do_fixed_add (f, op0, op1, true, sat_p); - break; case MULT_EXPR: gcc_assert (op0->mode == op1->mode); return do_fixed_multiply (f, op0, op1, sat_p); - break; case TRUNC_DIV_EXPR: gcc_assert (op0->mode == op1->mode); return do_fixed_divide (f, op0, op1, sat_p); - break; case LSHIFT_EXPR: return do_fixed_shift (f, op0, op1, true, sat_p); - break; case RSHIFT_EXPR: return do_fixed_shift (f, op0, op1, false, sat_p); - break; default: gcc_unreachable (); diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 2aa7c69e3f6..a2b9a96eced 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,5 +1,11 @@ 2016-09-27 Jakub Jelinek + * dependency.c (gfc_dep_compare_expr): Remove break after return. + * frontend-passes.c (optimize_op): Likewise. + * interface.c (gfc_current_interface_head): Likewise. + * symbol.c (check_conflict): Likewise. + * trans-intrinsic.c (build_fix_expr): Likewise. + PR fortran/77666 * trans-openmp.c (gfc_omp_private_outer_ref): Return true even for references to allocatable arrays. diff --git a/gcc/fortran/dependency.c b/gcc/fortran/dependency.c index 12e5d133151..8e78e430832 100644 --- a/gcc/fortran/dependency.c +++ b/gcc/fortran/dependency.c @@ -486,7 +486,6 @@ gfc_dep_compare_expr (gfc_expr *e1, gfc_expr *e2) case EXPR_FUNCTION: return gfc_dep_compare_functions (e1, e2, false); - break; default: return -2; diff --git a/gcc/fortran/frontend-passes.c b/gcc/fortran/frontend-passes.c index e049fb9f2ce..54b85544459 100644 --- a/gcc/fortran/frontend-passes.c +++ b/gcc/fortran/frontend-passes.c @@ -1491,7 +1491,6 @@ optimize_op (gfc_expr *e) case INTRINSIC_POWER: return optimize_power (e); - break; default: break; diff --git a/gcc/fortran/interface.c b/gcc/fortran/interface.c index 9a0ccf81185..5883e390d31 100644 --- a/gcc/fortran/interface.c +++ b/gcc/fortran/interface.c @@ -4306,16 +4306,13 @@ gfc_current_interface_head (void) { case INTERFACE_INTRINSIC_OP: return current_interface.ns->op[current_interface.op]; - break; case INTERFACE_GENERIC: case INTERFACE_DTIO: return current_interface.sym->generic; - break; case INTERFACE_USER_OP: return current_interface.uop->op; - break; default: gcc_unreachable (); diff --git a/gcc/fortran/symbol.c b/gcc/fortran/symbol.c index 3026356d111..cbe4347351f 100644 --- a/gcc/fortran/symbol.c +++ b/gcc/fortran/symbol.c @@ -464,7 +464,6 @@ check_conflict (symbol_attribute *attr, const char *name, locus *where) gfc_error ("Namelist group name at %L cannot have the " "SAVE attribute", where); return false; - break; case FL_PROCEDURE: /* Conflicts between SAVE and PROCEDURE will be checked at resolution stage, see "resolve_fl_procedure". */ diff --git a/gcc/fortran/trans-intrinsic.c b/gcc/fortran/trans-intrinsic.c index d3f684290bb..954f7b3afd4 100644 --- a/gcc/fortran/trans-intrinsic.c +++ b/gcc/fortran/trans-intrinsic.c @@ -410,19 +410,15 @@ build_fix_expr (stmtblock_t * pblock, tree arg, tree type, { case RND_FLOOR: return build_fixbound_expr (pblock, arg, type, 0); - break; case RND_CEIL: return build_fixbound_expr (pblock, arg, type, 1); - break; case RND_ROUND: return build_round_expr (arg, type); - break; case RND_TRUNC: return fold_build1_loc (input_location, FIX_TRUNC_EXPR, type, arg); - break; default: gcc_unreachable (); diff --git a/gcc/hsa.c b/gcc/hsa.c index ff978702489..c9c4957e425 100644 --- a/gcc/hsa.c +++ b/gcc/hsa.c @@ -947,7 +947,6 @@ hsa_internal_fn::get_arity () case IFN_SCALB: case IFN_LDEXP: return 2; - break; case IFN_CLRSB: case IFN_CLZ: case IFN_CTZ: diff --git a/gcc/objc/ChangeLog b/gcc/objc/ChangeLog index 6badc59df74..74dda97e39c 100644 --- a/gcc/objc/ChangeLog +++ b/gcc/objc/ChangeLog @@ -1,3 +1,8 @@ +2016-09-27 Jakub Jelinek + + * objc-act.c (continue_class): Remove break after return. + (objc_maybe_printable_name): Likewise. + 2016-08-12 Marek Polacek PR c/7652 diff --git a/gcc/objc/objc-act.c b/gcc/objc/objc-act.c index 44f01d2c9fe..374f1041aea 100644 --- a/gcc/objc/objc-act.c +++ b/gcc/objc/objc-act.c @@ -7054,7 +7054,6 @@ continue_class (tree klass) #endif /* OBJCPLUS */ return get_class_ivars (implementation_template, true); - break; } case CLASS_INTERFACE_TYPE: { @@ -7070,7 +7069,6 @@ continue_class (tree klass) pop_lang_context (); #endif /* OBJCPLUS */ return NULL_TREE; - break; } default: return error_mark_node; @@ -9279,7 +9277,6 @@ objc_maybe_printable_name (tree decl, int v ATTRIBUTE_UNUSED) { case FUNCTION_DECL: return objc_demangle (IDENTIFIER_POINTER (DECL_NAME (decl))); - break; /* The following happens when we are printing a deprecation warning for a method. The warn_deprecation() will end up @@ -9294,17 +9291,14 @@ objc_maybe_printable_name (tree decl, int v ATTRIBUTE_UNUSED) case INSTANCE_METHOD_DECL: case CLASS_METHOD_DECL: return IDENTIFIER_POINTER (DECL_NAME (decl)); - break; /* This happens when printing a deprecation warning for a property. We may want to consider some sort of pretty printing (eg, include the class name where it was declared ?). */ case PROPERTY_DECL: return IDENTIFIER_POINTER (PROPERTY_NAME (decl)); - break; default: return NULL; - break; } } diff --git a/gcc/rtlanal.c b/gcc/rtlanal.c index 470c143fcf2..2a0a1d2d99d 100644 --- a/gcc/rtlanal.c +++ b/gcc/rtlanal.c @@ -1244,7 +1244,6 @@ modified_between_p (const_rtx x, const rtx_insn *start, const rtx_insn *end) if (memory_modified_in_insn_p (x, insn)) return 1; return 0; - break; case REG: return reg_set_between_p (x, start, end); @@ -1299,7 +1298,6 @@ modified_in_p (const_rtx x, const_rtx insn) if (memory_modified_in_insn_p (x, insn)) return 1; return 0; - break; case REG: return reg_set_p (x, insn); diff --git a/gcc/trans-mem.c b/gcc/trans-mem.c index 2a6e101e9f2..6fc67fb1a22 100644 --- a/gcc/trans-mem.c +++ b/gcc/trans-mem.c @@ -165,7 +165,6 @@ get_attrs_for (const_tree x) { case FUNCTION_DECL: return TYPE_ATTRIBUTES (TREE_TYPE (x)); - break; default: if (TYPE_P (x)) diff --git a/gcc/tree-if-conv.c b/gcc/tree-if-conv.c index a57c1c5a25f..eec431e6a4f 100644 --- a/gcc/tree-if-conv.c +++ b/gcc/tree-if-conv.c @@ -1050,7 +1050,6 @@ if_convertible_stmt_p (gimple *stmt, vec refs) print_gimple_stmt (dump_file, stmt, 0, TDF_SLIM); } return false; - break; } return true; diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index 35c06b8ac4d..e07e93be30e 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -10215,7 +10215,6 @@ simplify_stmt_using_ranges (gimple_stmt_iterator *gsi) case MIN_EXPR: case MAX_EXPR: return simplify_min_or_max_using_ranges (stmt); - break; default: break;