re PR target/36425 (Option -mno-isel not working)

2008-06-11  Edmar Wienskoski  <edmar@freescale.com>

	PR target/36425
	* config/rs6000/rs6000.c (rs6000_override_options): Set
	rs6000_isel conditionally to the absence of comand line
	override.
	* config/rs6000/linuxspe.h (SUBSUBTARGET_OVERRIDE_OPTIONS):
	Remove duplicate rs6000_isel setting.
	* config/rs6000/eabispe.h: Ditto.

testsuite:
2008-06-11  Edmar Wienskoski  <edmar@freescale.com>

	* gcc.target/powerpc/e500-1.c: New test case to verify
	mno-isel option.

From-SVN: r136681

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 95a86b0d398cf4db490d7eab5d85543ddac5e601..d66d335dad96a108b4d37f27ab842b741cd8b489 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,13 @@
+2008-06-11  Edmar Wienskoski  <edmar@freescale.com>
+
+       PR target/36425
+       * config/rs6000/rs6000.c (rs6000_override_options): Set
+       rs6000_isel conditionally to the absence of comand line
+       override.
+       * config/rs6000/linuxspe.h (SUBSUBTARGET_OVERRIDE_OPTIONS):
+       Remove duplicate rs6000_isel setting.
+       * config/rs6000/eabispe.h: Ditto.
+
 2008-06-11  Richard Guenther  <rguenther@suse.de>
 
        * alias.c (get_alias_set): Use the element alias-set for arrays.

diff --git a/gcc/config/rs6000/eabispe.h b/gcc/config/rs6000/eabispe.h
index c3a3f2b4e53d713b0e91db88b23b4d147ec9902c..ae14651deba1818d8a5e855e4381b7c3501f3571 100644 (file)
--- a/gcc/config/rs6000/eabispe.h
+++ b/gcc/config/rs6000/eabispe.h
@@ -37,8 +37,6 @@
     rs6000_float_gprs = 1; \
   if (!rs6000_explicit_options.spe) \
     rs6000_spe = 1; \
-  if (!rs6000_explicit_options.isel) \
-    rs6000_isel = 1; \
   if (target_flags & MASK_64BIT) \
     error ("-m64 not supported in this configuration")
 

diff --git a/gcc/config/rs6000/linuxspe.h b/gcc/config/rs6000/linuxspe.h
index c526cf8dce49811d5943db7caca3c47b33c6db71..bcc403e2a3c257e3a523b0363bb853b7ff7f1b03 100644 (file)
--- a/gcc/config/rs6000/linuxspe.h
+++ b/gcc/config/rs6000/linuxspe.h
@@ -36,8 +36,6 @@
     rs6000_float_gprs = 1; \
   if (!rs6000_explicit_options.spe) \
     rs6000_spe = 1; \
-  if (!rs6000_explicit_options.isel) \
-    rs6000_isel = 1; \
   if (target_flags & MASK_64BIT) \
     error ("-m64 not supported in this configuration")
 

diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index dcbccbca38ea875d5134a65173de2f30a8730dc2..66e4b51773d41dae8f69cccc76b0f8e06c8ecc3d 100644 (file)
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -1553,7 +1553,7 @@ rs6000_override_options (const char *default_cpu)
        }
     }
 
-  if (TARGET_E500)
+  if (TARGET_E500 && !rs6000_explicit_options.isel)
     rs6000_isel = 1;
 
   if (rs6000_cpu == PROCESSOR_PPCE300C2 || rs6000_cpu == PROCESSOR_PPCE300C3)

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 08bd74ca62411c7c14008e4e87b9c41ba3b3937a..dfbe10d872cc8a13443174de2adc91a83c8e47dc 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2008-06-11  Edmar Wienskoski  <edmar@freescale.com>
+
+       PR target/36425
+       * gcc.target/powerpc/e500-1.c: New test case to verify
+       mno-isel option.
+
 2008-06-11  Eric Botcazou  <ebotcazou@adacore.com>
 
        * gnat.dg/specs/varsize_copy.ad[sb]: New test.

diff --git a/gcc/testsuite/gcc.target/powerpc/e500-1.c b/gcc/testsuite/gcc.target/powerpc/e500-1.c
new file mode 100644 (file)
index 0000000..76a0e4a
--- /dev/null
+++ b/gcc/testsuite/gcc.target/powerpc/e500-1.c
@@ -0,0 +1,14 @@
+/* Test functioning of command option -mno-isel */
+/* { dg-do compile { target powerpc*-*-linux* } } */
+/* { dg-options "-O2 -mno-isel" } */
+
+/* { dg-final { scan-assembler-not "isel" } } */
+
+int
+foo (int x, int y)
+{
+  if (x < y)
+    return x;
+  else
+    return y;
+}