This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Update readelf to know about the new ELF constants


Hi Uli,

: >   #define EM_AVR        0x1057   /* Atmel AVR 8-bit microcontroller.  */
: >   #define EM_FR30       0x3330   /* Fujitsu FR30.  */
: >   #define EM_D10V       0x7650   /* Mitsubishi D10V.  */
: >   #define EM_D30V       0x7676   /* Mitsubishi D30V.  */
: >   #define EM_ALPHA      0x9026   /* DEC Alpha.  */
: >   #define EM_V850       0x9080   /* NEC v850.  */
: >   #define EM_ARC        0x9040   /* ARC Cores.  */
: >   #define EM_M32R       0x9041   /* Mitsubishi M32R.  */
: >   #define EM_MN10300    0xbeef   /* Matsushita MN10300.  */
: >   #define EM_MN10200    0xdead   /* Matsushita MN10200.  */
: 
: I''m not sure this will be OK.  All the numbers allocates are in
: sequence (with a few holes left).

Is there a good reason for this ?  These are just numbers right ?

The values mentioned above are already in use in real code, so
changing them would not be a good idea.  Unless there is a real need
to have consecutive EM_ values, I would prefer to leave them alone.

: Besides, there is already a EM_ALPHA.

Except that it is not used, at least not by the binutils code.
Would you agree to calling it:

 #define EM_ALPHA_ALTERNATE     0x9026  /* An alternate DEC Alpha machine number.  */

Cheers
	Nick

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]