This is the mail archive of the binutils@sourceware.org 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]
Other format: [Raw text]

Re: [PATCH] GAS: Add ECOFF `.aent' pseudo-op support


Hi Maciej,

> Assembly language documentation available describes the pseudo-op 
> respectively as follows[1]:
> 
> "
> .aent name, symno Sets an alternate entry point for the current
>                   procedure.  Use this information when you want
>                   to generate information for the debugger.  It must
>                   appear inside an .ent/.end pair."
> 
> and[2]:
> 
> "
> .aent name [,symno]
>      Sets an alternate entry point for the current procedure.  Use this
>      information when you want to generate information for the debugger.
>      This directive must appear between a pair of .ent and .end directives.
>      (The optional symno is for compiler use only.  It refers to a dense
>      number in a .T file (symbol table).)"

Should this documentation be added to gas/doc/as.texinfo ?  Or failing that
maybe names of the pseudo ops could be added along with the URLs in [1] and [2] ?

> 	gas/
> 	* ecoff.c (ecoff_directive_ent, add_procedure): Handle `.aent'.
> 	* config/obj-ecoff.c (obj_pseudo_table): Add "aent" entry.  
> 	* config/obj-elf.c (ecoff_debug_pseudo_table): Likewise.
> 	* testsuite/gas/mips/aent-2.d: New test.
> 	* testsuite/gas/mips/aent-mdebug.d: New test.
> 	* testsuite/gas/mips/aent-mdebug-2.d: New test.
> 	* testsuite/gas/mips/mips.exp: Run the new tests.
 
Approved - please apply.

Is this feature worth an entry in the gas/NEWS file ?  I am not an aficionado of
the ECOFF format, so I am not sure of the significance of the feat


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