This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [binutils-gdb] Add .nop assembler directive
On Mon, Feb 19, 2018 at 2:03 PM, Alan Modra <amodra@gmail.com> wrote:
> On Mon, Feb 19, 2018 at 06:10:19AM -0800, H.J. Lu wrote:
>> On Sun, Feb 18, 2018 at 8:50 PM, Alan Modra <amodra@gmail.com> wrote:
>> > On Sat, Feb 17, 2018 at 01:22:41PM -0000, H.J.Lu wrote:
>> >> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=62a02d25b6e5d9f92c205260daa11355d0c62532
>> >>
>> >> commit 62a02d25b6e5d9f92c205260daa11355d0c62532
>> >> Author: H.J. Lu <hjl.tools@gmail.com>
>> >> Date: Sat Feb 17 05:20:42 2018 -0800
>> >>
>> >> Add .nop assembler directive
>> >
>> > It looks like this patch caused the following regressions. Please
>> > investigate.
>> >
>> > m68hc11-elf +FAIL: objdump -WL
> [snip lots more]
>>
>> These are caused by
>>
>> config/tc-m68hc11.h:#define NO_PSEUDO_DOT 1
>> config/tc-m68k.h:#define NO_PSEUDO_DOT 1
>> config/tc-spu.h:#define NO_PSEUDO_DOT 1
>> config/tc-xgate.h:#define NO_PSEUDO_DOT 1
>> config/tc-xtensa.c:/* NO_PSEUDO_DOT hook */
>> config/tc-xtensa.h:#define NO_PSEUDO_DOT xtensa_check_inside_bundle ()
>> config/tc-z80.h:#define NO_PSEUDO_DOT 1
>>
>> We match .nop diretive to nop instruction. This patch avoids
>> matching .nop diretive to nop instruction when NO_PSEUDO_DOT
>> is defined to 1. OK for master?
>
>> * read.c (potable): Sort nop.
>> (read_a_source_file): Avoid matching .nop diretive to nop
>> instruction.
>
> Another fix would be to add "nop" with a NULL poc_handler to each of
> these targets' md_pseudo_table, avoiding any extra code. However, I'm
> inclined to think the directive should be renamed.
The extra codes are only applies to NO_PSEUDO_DOT targets.
--
H.J.