------- Additional Comments From amodra at bigpond dot net dot au  2005-09-07 
01:26 -------
Indeed, that is what is happening.  -m64 ought to normally imply
-mpowerpc-gfxopt, because all powerpc64 capable chips also support the insns
enabled by -mpowerpc-gfxopt as far as I know.  However, I guess there's not much
harm in preventing -m64 from overriding a prior -mno-powerpc-gfxopt.

Index: gcc/config/rs6000/rs6000.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/rs6000/rs6000.c,v
retrieving revision 1.866
diff -u -p -r1.866 rs6000.c
--- gcc/config/rs6000/rs6000.c  6 Sep 2005 02:04:59 -0000       1.866
+++ gcc/config/rs6000/rs6000.c  7 Sep 2005 01:11:56 -0000
@@ -1620,9 +1620,9 @@ rs6000_handle_option (size_t code, const
 #else
     case OPT_m64:
 #endif
-      target_flags |= MASK_POWERPC64 | MASK_POWERPC | MASK_PPC_GFXOPT;
-      target_flags_explicit |= MASK_POWERPC64 | MASK_POWERPC
-       | MASK_PPC_GFXOPT;
+      target_flags |= MASK_POWERPC64 | MASK_POWERPC;
+      target_flags |= ~target_flags_explicit & MASK_PPC_GFXOPT;
+      target_flags_explicit |= MASK_POWERPC64 | MASK_POWERPC;
       break;
 
 #ifdef TARGET_USES_AIX64_OPT


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23704

Reply via email to