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

On Fri, Feb 16, 2018 at 6:59 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Fri, Feb 16, 2018 at 4:49 PM, Alan Modra <amodra@gmail.com> wrote:
>> On Fri, Feb 16, 2018 at 08:07:50AM -0800, H.J. Lu wrote:
>>> Good points.  Here is the updated patch.   OK for master?
>>
>> OK, except for one quibble with the default md_generate_nops.  I think
>> that should error rather than filling with zeros (which is likely to
>> be an illegal instruction or trap).
>>
>
> Fixed.  This is the patch I will check in tomorrow.
>

Here is a patch for gas/NEWS.

diff --git a/gas/NEWS b/gas/NEWS
index a591a7b200..27ee306f2f 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -1,5 +1,8 @@
 -*- text -*-

+* Add support for .nop directive.  It is currently supported only for
+  x86 targets.
+
 Changes in 2.30:

 * Add support for loaction views in DWARF debug line information.


-- 
H.J.
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]