diff mbox

[X86] Fix up i386 option translations (PR translation/78922)

Message ID 20161227134608.GJ21933@tucnak
State New
Headers show

Commit Message

Jakub Jelinek Dec. 27, 2016, 1:46 p.m. UTC
Hi!

r201645 apparently added a useless file that nothing sources in
in a format incompatible with all other *.opt files (comments in *.opt
files start with ;, and no preprocessing is performed), the only thing that
it affects  is that all *.opt files from the tree are parsed to extract
translations from it, so we have stuff like:
#: config/i386/stringop.opt:8
msgid "the Free Software Foundation; either version 3, or (at your option)"
msgstr ""

#: config/i386/stringop.opt:13
msgid "MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the"
msgstr ""

#: config/i386/stringop.opt:18
msgid "see <http://www.gnu.org/licenses/>.  */"
msgstr ""

The following patch just removes it.  Bootstrapped/regtested on x86_64-linux
and i686-linux, ok for trunk/branches?

2016-12-27  Jakub Jelinek  <jakub@redhat.com>

	PR translation/78922
	* config/i386/stringop.opt: Remove.


	Jakub

Comments

Uros Bizjak Dec. 27, 2016, 2:31 p.m. UTC | #1
On Tue, Dec 27, 2016 at 2:46 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> Hi!
>
> r201645 apparently added a useless file that nothing sources in
> in a format incompatible with all other *.opt files (comments in *.opt
> files start with ;, and no preprocessing is performed), the only thing that
> it affects  is that all *.opt files from the tree are parsed to extract
> translations from it, so we have stuff like:
> #: config/i386/stringop.opt:8
> msgid "the Free Software Foundation; either version 3, or (at your option)"
> msgstr ""
>
> #: config/i386/stringop.opt:13
> msgid "MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the"
> msgstr ""
>
> #: config/i386/stringop.opt:18
> msgid "see <http://www.gnu.org/licenses/>.  */"
> msgstr ""
>
> The following patch just removes it.  Bootstrapped/regtested on x86_64-linux
> and i686-linux, ok for trunk/branches?
>
> 2016-12-27  Jakub Jelinek  <jakub@redhat.com>
>
>         PR translation/78922
>         * config/i386/stringop.opt: Remove.

OK everywhere as obvious.

Thanks,
Uros.

> --- gcc/config/i386/stringop.opt.jj     2016-01-04 14:55:55.000000000 +0100
> +++ gcc/config/i386/stringop.opt        2016-12-26 10:45:57.548373327 +0100
> @@ -1,31 +0,0 @@
> -/* Definitions for stringop option handling for IA-32.
> -   Copyright (C) 2013-2016 Free Software Foundation, Inc.
> -
> -This file is part of GCC.
> -
> -GCC is free software; you can redistribute it and/or modify
> -it under the terms of the GNU General Public License as published by
> -the Free Software Foundation; either version 3, or (at your option)
> -any later version.
> -
> -GCC is distributed in the hope that it will be useful,
> -but WITHOUT ANY WARRANTY; without even the implied warranty of
> -MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -GNU General Public License for more details.
> -
> -You should have received a copy of the GNU General Public License
> -along with GCC; see the files COPYING3.  If not,
> -see <http://www.gnu.org/licenses/>.  */
> -
> -Enum(stringop_alg) String(rep_byte) Value(rep_prefix_1_byte)
> -
> -#undef DEF_ENUM
> -#define DEF_ENUM EnumValue
> -
> -#undef DEF_ALG
> -#define DEF_ALG(alg, name) Enum(stringop_alg) String(name) Value(alg)
> -
> -#include "stringop.def"
> -
> -#undef DEF_ENUM
> -#undef DEF_ALG
>
>         Jakub
diff mbox

Patch

--- gcc/config/i386/stringop.opt.jj	2016-01-04 14:55:55.000000000 +0100
+++ gcc/config/i386/stringop.opt	2016-12-26 10:45:57.548373327 +0100
@@ -1,31 +0,0 @@ 
-/* Definitions for stringop option handling for IA-32.
-   Copyright (C) 2013-2016 Free Software Foundation, Inc.
-
-This file is part of GCC.
-
-GCC is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 3, or (at your option)
-any later version.
-
-GCC is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
-
-You should have received a copy of the GNU General Public License
-along with GCC; see the files COPYING3.  If not,
-see <http://www.gnu.org/licenses/>.  */
-
-Enum(stringop_alg) String(rep_byte) Value(rep_prefix_1_byte)
-
-#undef DEF_ENUM
-#define DEF_ENUM EnumValue
-
-#undef DEF_ALG
-#define DEF_ALG(alg, name) Enum(stringop_alg) String(name) Value(alg)
-
-#include "stringop.def"
-
-#undef DEF_ENUM
-#undef DEF_ALG