Prefer object over notype symbols when disassembling

Alan Modra amodra@gmail.com
Mon Jan 6 22:23:00 GMT 2020


On Mon, Jan 06, 2020 at 02:11:23PM +0000, Maciej W. Rozycki wrote:
> On Tue, 17 Dec 2019, Alan Modra wrote:
> 
> > diff --git a/ld/testsuite/ld-mips-elf/eh-frame5.s b/ld/testsuite/ld-mips-elf/eh-frame5.s
> > index 741a021685..cf5a11c37a 100644
> > --- a/ld/testsuite/ld-mips-elf/eh-frame5.s
> > +++ b/ld/testsuite/ld-mips-elf/eh-frame5.s
> > @@ -2,6 +2,7 @@
> >  	.cfi_personality 0x0,local_pers
> >  	.cfi_lsda 0x0,LSDA
> >  	.ent	f1
> > +	.type	f1, @function
> >  f1:
> >  	nop
> >  	.end	f1
> 
>  If this and similar changes throughout are required, then we have a bug 
> somewhere, probably in GAS, because `.ent' is supposed to set STT_FUNCTION 
> for the symbol named, making a `.type' pseudo-op to do the same redundant.  
> Cf. `s_mips_ent' and:
> 
>   symbol_get_bfdsym (symbolP)->flags |= BSF_FUNCTION;
> 
> I'd rather keep the old arrangement in the test suite to provide coverage 
> for this semantics.

I'll admit to going wild and adding .type @function in lots of places
after finding it was needed for some mips tests on mips-sgi-irix.  It
looks like the following files should have my change reverted.  I'll
test and commit.

ld/testsuite/ld-mips-elf/eh-frame5.s
ld/testsuite/ld-mips-elf/ehdr_start-new.s
ld/testsuite/ld-mips-elf/ehdr_start-o32.s
ld/testsuite/ld-mips-elf/mips16-call-global-1.s
ld/testsuite/ld-mips-elf/mips16-intermix-1.s
ld/testsuite/ld-mips-elf/mips16-pic-1b.s
ld/testsuite/ld-mips-elf/mips16-pic-4c.s
ld/testsuite/ld-mips-elf/no-shared-1-n64.s
ld/testsuite/ld-mips-elf/no-shared-1-o32.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-1b-micromips.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-1b.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-2a.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-3b.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-4b.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-5a.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-6-n32c.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-6-n64c.s
ld/testsuite/ld-mips-elf/pic-and-nonpic-6-o32c.s
ld/testsuite/ld-mips-elf/pie.s
ld/testsuite/ld-mips-elf/relax-jalr.s

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list