RFA: new command line switch for MIPS assembler

Ulf Carlsson ulfc@calypso.engr.sgi.com
Fri Jul 14 14:29:00 GMT 2000


Hi Nick,

 > 2000-07-14  Nick Clifton  <nickc@cygnus.com>
 > 
 > 	* config/tc-mips.c (mips_disable_float_construction): New
 > 	static variable.  Set to true if doubles should not be
 > 	constructed by loading two single width fp registers with
 > 	halves of the value.
 > 	(mips_ip): Test mips_disable_float_construction.
 > 	(md_longopts): Add command line switches --construct-floats
 > 	and --no-construct-floats.
 > 	(md_parse_option): Parse new command line options.
 > 	(md_show_usage): Describe new command line options.
 > 
 > 	* doc/c-mips.texi: Document new command line options.

This looks fine with a minor exception:

 > *************** struct option md_longopts[] = {
 > *** 8866,8871 ****
 > --- 8875,8886 ----
 >   #define OPTION_NO_M7000_HILO_FIX (OPTION_MD_BASE + 40)
 >     {"no-fix-7000", no_argument, NULL, OPTION_NO_M7000_HILO_FIX},
 >   
 > + #define OPTION_CONSTRUCT_FLOATS (OPTION_MD_BASE + 41)
 > +   {"construct-floats", no_argument, NULL, OPTION_CONSTRUCT_FLOATS},
 > + 
 > + #define OPTION_NO_CONSTRUCT_FLOATS (OPTION_MD_BASE + 42)
 > +   {"no-construct-floats", no_argument, NULL, OPTION_NO_CONSTRUCT_FLOATS},
 > + 
 >   #define OPTION_CALL_SHARED (OPTION_MD_BASE + 7)
 >   #define OPTION_NON_SHARED (OPTION_MD_BASE + 8)
 >   #define OPTION_XGOT (OPTION_MD_BASE + 19)

If you look further down in the list you'll see that 41 and 42 already
are used by OPTION_GP32 and OPTION_GP64.  I think 43 and 44 should be
a safe bet.

Ulf


More information about the Binutils mailing list