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] x86: Add .nop directive to assembler

From: Andrew Cooper <andrew dot cooper3 at citrix dot com>
To: "H.J. Lu" <hjl dot tools at gmail dot com>, Jan Beulich <JBeulich at suse dot com>
Cc: "Maciej W. Rozycki" <macro at mips dot com>, Binutils <binutils at sourceware dot org>
Date: Wed, 14 Feb 2018 17:03:13 +0000
Subject: Re: [PATCH] x86: Add .nop directive to assembler
Authentication-results: sourceware.org; auth=none
References: <CAMe9rOr9n4XwM4wMhR3RNP4EXUE37nu0ui3SL4-nUvLAF_T9yA@mail.gmail.com> <5A82B4CC02000078001A734A@prv-mh.provo.novell.com> <CAMe9rOqC-69okYQTNOJYcjsVF2LuAuKNV1-Ydh5Zb+m77Pzo_g@mail.gmail.com>

On 14/02/18 14:03, H.J. Lu wrote:
> On Tue, Feb 13, 2018 at 12:50 AM, Jan Beulich <JBeulich@suse.com> wrote:
>
>> Finally, would you mind making the diagnostic complaining about too
>> large a LIMIT also report the upper bound (not the least because - as
>> per above remark - this may change over time)?
>>
> Done.
>
> Here is the updated patch.  OK for master?

Sorry for the delay.  This patch does work correctly.

As for exactly how this functionality gets implemented, I am not fussed
at all.  I'll defer that to the binutils community - its very easy for
me to tweak my patch series to start using it when the eventual
implementation is decided upon.

~Andrew

References:
- [PATCH] x86: Add .nop directive to assembler
  - From: H.J. Lu
- Re: [PATCH] x86: Add .nop directive to assembler
  - From: Jan Beulich
- Re: [PATCH] x86: Add .nop directive to assembler
  - From: H.J. Lu

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