> Am 22.03.2025 um 07:59 schrieb Jakub Jelinek <ja...@redhat.com>:
>
> Hi!
>
> Given the recent PR119406 I've tried to grep for concatenated string
> literals without space at the end of one line and at the start of next line,
> unless it was obviously intentional.
> Furthermore, I've then looked through gcc.pot looking for 2 adjacent spaces
> and looking back if that wasn't the case of "something "
> " with spaces at both sides".
>
> Here is the result from that.
>
> I think just the c.opt change needs an explanation, the "" in the
> description is simply eaten up somewhere during the option processing and
> gcc -v --help before this patch was displaying
> -Wdeprecated-literal-operator Warn about deprecated space between and
> suffix in a user-defined literal operator.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
Ok
Richard
> 2025-03-22 Jakub Jelinek <ja...@redhat.com>
>
> gcc/
> * gimplify.cc (warn_switch_unreachable_and_auto_init_r): Add missing
> space in the middle of diagnostics.
> * tree-vect-stmts.cc (vectorizable_load): Add missing space in the
> middle of debug dump message.
> * syum-exec/sym-exec-state.cc (state::check_args_compatibility):
> Likewise.
> gcc/c-family/
> * c.opt (Wdeprecated-literal-operator): Use \"\" rather than ""
> in option description.
> gcc/fortran/
> * resolve.cc (resolve_procedure_expression): Remove extraneous space
> from the middle of diagnostics.
>
> --- gcc/gimplify.cc.jj 2025-03-21 20:25:38.457063450 +0100
> +++ gcc/gimplify.cc 2025-03-21 22:10:39.972918734 +0100
> @@ -2395,7 +2395,7 @@ warn_switch_unreachable_and_auto_init_r
> const char *var_name_str = TREE_STRING_POINTER (var_name);
>
> warning_at (gimple_location (stmt), OPT_Wtrivial_auto_var_init,
> - "%qs cannot be initialized with"
> + "%qs cannot be initialized with "
> "%<-ftrivial-auto-var_init%>",
> var_name_str);
> break;
> --- gcc/tree-vect-stmts.cc.jj 2025-03-14 15:29:30.223916643 +0100
> +++ gcc/tree-vect-stmts.cc 2025-03-21 22:15:29.473965423 +0100
> @@ -12576,7 +12576,7 @@ vectorizable_load (vec_info *vinfo,
>
> if (dump_enabled_p ())
> dump_printf_loc (MSG_NOTE, vect_location,
> - "vect_model_load_cost:"
> + "vect_model_load_cost: "
> "strided group_size = %d .\n",
> group_size);
> }
> --- gcc/sym-exec/sym-exec-state.cc.jj 2025-01-02 11:47:11.796479462 +0100
> +++ gcc/sym-exec/sym-exec-state.cc 2025-03-21 22:20:36.036778243 +0100
> @@ -265,7 +265,7 @@ state::check_args_compatibility (tree ar
> || TREE_CODE (arg2) == INTEGER_CST))
> {
> if (dump_file && (dump_flags & TDF_DETAILS))
> - fprintf (dump_file, "Sym-Exec: Incompatible destination"
> + fprintf (dump_file, "Sym-Exec: Incompatible destination "
> "and argument sizes.\n");
>
> return false;
> --- gcc/c-family/c.opt.jj 2025-03-07 23:59:21.000000000 +0100
> +++ gcc/c-family/c.opt 2025-03-21 22:32:54.943680670 +0100
> @@ -666,7 +666,7 @@ type and the other is of a floating-poin
>
> Wdeprecated-literal-operator
> C++ ObjC++ Var(warn_deprecated_literal_operator) Warning
> -Warn about deprecated space between "" and suffix in a user-defined literal
> operator.
> +Warn about deprecated space between \"\" and suffix in a user-defined
> literal operator.
>
> Wdeprecated-non-prototype
> C ObjC Var(warn_deprecated_non_prototype) Init(-1) Warning
> --- gcc/fortran/resolve.cc.jj 2025-03-21 20:25:38.352064880 +0100
> +++ gcc/fortran/resolve.cc 2025-03-21 22:26:07.660246428 +0100
> @@ -1986,7 +1986,7 @@ resolve_procedure_expression (gfc_expr*
> if (is_illegal_recursion (sym, gfc_current_ns))
> {
> if (sym->attr.use_assoc && expr->symtree->name[0] == '@')
> - gfc_warning (0, "Non-RECURSIVE procedure %qs from module %qs is "
> + gfc_warning (0, "Non-RECURSIVE procedure %qs from module %qs is"
> " possibly calling itself recursively in procedure %qs. "
> " Declare it RECURSIVE or use %<-frecursive%>",
> sym->name, sym->module, gfc_current_ns->proc_name->name);
>
> Jakub
>
