On Thu, Oct 18, 2012 at 09:08:34PM +0200, Dominique Dhumieres wrote: > > 2012-10-17 Michael Meissner <meiss...@linux.vnet.ibm.com> > > > > * config/rs6000/rs6000.opt (rs6000_isa_flags): New flag word to > > replace target_flags that gives us 63 possible switches. > > (x_rs6000_isa_flags): Save area for rs6000_isa_flags. > > (x_rs6000_isa_flags_explicit): Save area for > > ... > > Bootstrap failed on powerpc-apple-darwin9 with
Just to let you know, I'm iterating on a patch right now. There were 3 bugs with my checkin: 1) The libobjc/encoding.c issue that Eric ran into. The file encoding.c includes compiler tm.h files, and then redefines stuff so that it doesn't need the target option structure. I have a bandaid to redefine TARGET_VSX, TARGET_ALTIVEC, and TARGET_64BIT, but this file really cries out for an Objective C user to reengineer and rewrite it so that it doesn't have to include compiler internal files. 2) The redefinition problem that Dominique found. This was caused by moving POWERPC_MASKS from rs6000.c to rs6000-cpus.def, where it more logically belongs. I think the right solution is to move the #ifdef to rs6000-cpus.def instead of keeping it in rs6000.c. 3) I forgot to update the -mdebug=reg support so it prints out the current options once I swtiched from using target_flags to rs6000_isa_flags. I discovered this today, when doing some future work (that needs new options). I'll send out the patch if my bootstrap finishes. -- Michael Meissner, IBM 5 Technology Place Drive, M/S 2757, Westford, MA 01886-3141, USA meiss...@linux.vnet.ibm.com fax +1 (978) 399-6899
