Patchwork rs6000_handle_option global state avoidance, part 1

login
register
mail settings
Submitter Michael Meissner
Date May 5, 2011, 9:55 p.m.
Message ID <20110505215529.GA17845@hungry-tiger.westford.ibm.com>
Download mbox | patch
Permalink /patch/94322/
State New
Headers show

Comments

Michael Meissner - May 5, 2011, 9:55 p.m.
On Thu, May 05, 2011 at 03:08:43PM -0400, Michael Meissner wrote:
> Yes, it was an error on my part.  Sorry.  I will fix the GCC 4.6 branch.  I
> will hold off fixing the on trunk, on the assumption this patch will go in.  I
> can fix it if desired.

I committed the following patch on the GCC 4.6 branch (GCC 4.5 and earlier is
not affected):

2011-05-05  Michael Meissner  <meissner@linux.vnet.ibm.com>

	* config/rs6000/rs6000.c (rs6000_handle_option): Add missing break
	for OPT_mfpu_ case.

Patch

Index: gcc/config/rs6000/rs6000.c
===================================================================
--- gcc/config/rs6000/rs6000.c	(revision 173455)
+++ gcc/config/rs6000/rs6000.c	(working copy)
@@ -4630,6 +4630,7 @@  rs6000_handle_option (size_t code, const
         target_flags_explicit |= MASK_SOFT_FLOAT;
         rs6000_single_float = rs6000_double_float = 0;
       }
+      break;
 
     case OPT_mrecip:
       rs6000_recip_name = (value) ? "default" : "none";