Re: [PATCH 1/4] Remove DEF_ENUM from stringop.def.

2021-04-21 Thread Uros Bizjak via Gcc-patches
On Wed, Apr 21, 2021 at 10:28 AM Martin Liska  wrote:
>
>
> gcc/ChangeLog:
>
> * config/i386/i386-options.c (DEF_ENUM): Remove it.
> * config/i386/i386-opts.h (DEF_ENUM): Likewise.
> * config/i386/stringop.def (DEF_ENUM): Likewise.

OK.

Thanks,
Uros.

> ---
>  gcc/config/i386/i386-options.c | 2 --
>  gcc/config/i386/i386-opts.h| 4 
>  gcc/config/i386/stringop.def   | 9 -
>  3 files changed, 15 deletions(-)
>


[PATCH 1/4] Remove DEF_ENUM from stringop.def.

2021-04-21 Thread Martin Liska

gcc/ChangeLog:

* config/i386/i386-options.c (DEF_ENUM): Remove it.
* config/i386/i386-opts.h (DEF_ENUM): Likewise.
* config/i386/stringop.def (DEF_ENUM): Likewise.
---
 gcc/config/i386/i386-options.c | 2 --
 gcc/config/i386/i386-opts.h| 4 
 gcc/config/i386/stringop.def   | 9 -
 3 files changed, 15 deletions(-)

diff --git a/gcc/config/i386/i386-options.c b/gcc/config/i386/i386-options.c
index 7e59ccd988d..154234a6a44 100644
--- a/gcc/config/i386/i386-options.c
+++ b/gcc/config/i386/i386-options.c
@@ -1455,10 +1455,8 @@ ix86_valid_target_attribute_p (tree fndecl,
 }
 
 const char *stringop_alg_names[] = {
-#define DEF_ENUM
 #define DEF_ALG(alg, name) #name,
 #include "stringop.def"
-#undef DEF_ENUM
 #undef DEF_ALG
 };
 
diff --git a/gcc/config/i386/i386-opts.h b/gcc/config/i386/i386-opts.h
index de6e7e01661..04e4ad608fb 100644
--- a/gcc/config/i386/i386-opts.h
+++ b/gcc/config/i386/i386-opts.h
@@ -28,16 +28,12 @@ see the files COPYING3 and COPYING.RUNTIME respectively.  If not, see
 /* Algorithm to expand string function with.  */
 enum stringop_alg
 {
-#undef DEF_ENUM
-#define DEF_ENUM
-
 #undef DEF_ALG
 #define DEF_ALG(alg, name) alg, 
 
 #include "stringop.def"
 last_alg
 
-#undef DEF_ENUM
 #undef DEF_ALG
 };
 
diff --git a/gcc/config/i386/stringop.def b/gcc/config/i386/stringop.def
index 76898d26435..cd34b7e6b7d 100644
--- a/gcc/config/i386/stringop.def
+++ b/gcc/config/i386/stringop.def
@@ -17,21 +17,12 @@ You should have received a copy of the GNU General Public License
 along with GCC; see the files COPYING3.  If not,
 see .  */
 
-DEF_ENUM
 DEF_ALG (no_stringop, no_stringop)
-DEF_ENUM
 DEF_ALG (libcall, libcall)
-DEF_ENUM
 DEF_ALG (rep_prefix_1_byte, rep_byte)
-DEF_ENUM
 DEF_ALG (rep_prefix_4_byte, rep_4byte)
-DEF_ENUM
 DEF_ALG (rep_prefix_8_byte, rep_8byte)
-DEF_ENUM
 DEF_ALG (loop_1_byte, byte_loop)
-DEF_ENUM
 DEF_ALG (loop, loop)
-DEF_ENUM
 DEF_ALG (unrolled_loop, unrolled_loop)
-DEF_ENUM
 DEF_ALG (vector_loop, vector_loop)