i386.c (ix86_option_override_internal): Enable flag_ree also for 32bit targets.

	* config/i386/i386.c (ix86_option_override_internal): Enable flag_ree
	also for 32bit targets.

testsuite/ChangeLog:

	* gcc.dg/pr50038.c: Move to ...
	* gcc.target/i386/pr50038.c: ... here.  Test on all targets.

From-SVN: r182605

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 31210c0aa47e7f80d41a17749635c5f21c656c44..df3198885ba40157b381d1ef69712cbf5a2e45a7 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2011-12-21  Uros Bizjak  <ubizjak@gmail.com>
+
+       * config/i386/i386.c (ix86_option_override_internal): Enable flag_ree
+       also for 32bit targets.
+
 2011-12-21  Richard Henderson  <rth@redhat.com>
 
        PR target/51552

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index c913c13d479fbfe0836f525f721eff99d2e9a3b7..af075242e3766ce36d86966042787bd8a45fec76 100644 (file)
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -3445,12 +3445,14 @@ ix86_option_override_internal (bool main_args_p)
 #define USE_X86_64_FRAME_POINTER 0
 #endif
 
+  /* Enable redundant extension instructions removal at -O2 and higher.  */
+  if (optimize >= 2 && !global_options_set.x_flag_ree)
+    flag_ree = 1;
+
   /* Set the default values for switches whose default depends on TARGET_64BIT
      in case they weren't overwritten by command line options.  */
   if (TARGET_64BIT)
     {
-      if (optimize > 1 && !global_options_set.x_flag_ree)
-       flag_ree = 1;
       if (optimize >= 1 && !global_options_set.x_flag_omit_frame_pointer)
        flag_omit_frame_pointer = !USE_X86_64_FRAME_POINTER;
       if (flag_asynchronous_unwind_tables == 2)

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 1592e1314083689c440bd8e50e7df885b8b725e4..492a0c38cca123d0d0aea95cd83f85fb0818d154 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2011-12-21  Uros Bizjak  <ubizjak@gmail.com>
+
+       * gcc.dg/pr50038.c: Move to ...
+       * gcc.target/i386/pr50038.c: ... here.  Test on all targets.
+
 2011-12-21  Jason Merrill  <jason@redhat.com>
 
        PR c++/51611

diff --git a/gcc/testsuite/gcc.dg/pr50038.c b/gcc/testsuite/gcc.dg/pr50038.c
deleted file mode 100644 (file)
index 4d1c8f7..0000000
--- a/gcc/testsuite/gcc.dg/pr50038.c
+++ /dev/null
@@ -1,20 +0,0 @@
-/* PR target/50038 */
-/* { dg-do compile { target x86_64-*-* } } */
-/* { dg-options "-O2" } */
-
-void pr50038(int len, unsigned char *in, unsigned char *out)
-{
-  int i;
-  unsigned char xr, xg;
-  unsigned char xy=0;
-  for (i = 0; i < len; i++)
-    {
-      xr = *in++;
-      xg = *in++;
-      xy = (unsigned char) ((19595*xr + 38470*xg) >> 16);
-
-      *out++ = xy;
-    }
-}
-
-/* { dg-final { scan-assembler-times "movzbl" 2 } } */

diff --git a/gcc/testsuite/gcc.target/i386/pr50038.c b/gcc/testsuite/gcc.target/i386/pr50038.c
new file mode 100644 (file)
index 0000000..e111574
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/pr50038.c
@@ -0,0 +1,20 @@
+/* PR target/50038 */
+/* { dg-options "-O2" } */
+
+void
+test (int len, unsigned char *in, unsigned char *out)
+{
+  int i;
+  unsigned char xr, xg;
+  unsigned char xy=0;
+  for (i = 0; i < len; i++)
+    {
+      xr = *in++;
+      xg = *in++;
+      xy = (unsigned char) ((19595 * xr + 38470 * xg) >> 16);
+
+      *out++ = xy;
+    }
+}
+
+/* { dg-final { scan-assembler-times "movzbl" 2 } } */