[PATCH] x86: Add .nop directive to assembler

Andrew Cooper andrew.cooper3@citrix.com
Fri Feb 16 18:01:00 GMT 2018


On 16/02/18 16:07, H.J. Lu wrote:
> On Fri, Feb 16, 2018 at 7:41 AM, Maciej W. Rozycki <macro@mips.com> wrote:
>> On Fri, 16 Feb 2018, H.J. Lu wrote:
>>
>>> All fixed.  Here is the updated patch.  OK for master?
>>  I skimmed over briefly only.  I think the convention for the manual is to
>> use the generic term `no-op instruction' rather than `@code{NOP}'.
>>
>>  Firstly, not all targets we support have an actual NOP mnemonic.  And
>> even with x86 the encodings you choose are not true architectural NOP
>> instructions, but just machine code encodings which happen to execute
>> quickly and with no visible effects (and e.g. for MIPS `addiu $0, $3, -1'
>> yields no observable effect, however nobody calls that `nop'; there are
>> millions of such MIPS machine code encodings).
> Good points.  Here is the updated patch.   OK for master?

I've tested this latest patch, and everything looks ok.

~Andrew



More information about the Binutils mailing list