gas: fix rs_fill_nop listing

In commit a0094f1a70e1 ("gas: make .nops output visible in listing") I
was wrongly assuming fr_fix would be zero for rs_fill_nop, when that's
only a side effect of listing_newline() inserting dummy frags, but only
when file/line did actually change from the previous invocation. This is
in particular not going to be true when the .nops directive isn't the
first statement on a line.

diff --git a/gas/listing.c b/gas/listing.c
index 11ae450abf594175be9c1e7c855fecdddec45d9e..be71597525c411cc6ec1532ea53b8b8a868983c8 100644 (file)
--- a/gas/listing.c
+++ b/gas/listing.c
@@ -820,9 +820,8 @@ calc_hex (list_info_type *list)
        }
       else if (frag_ptr->fr_type == rs_fill_nop && frag_ptr->fr_opcode)
        {
-         gas_assert (!octet_in_frag);
-
          /* Print as many bytes from fr_opcode as is sensible.  */
+         octet_in_frag = 0;
          while (octet_in_frag < (unsigned int) frag_ptr->fr_offset
                 && data_buffer_size < MAX_BYTES - 3)
            {

diff --git a/gas/testsuite/gas/i386/i386.exp b/gas/testsuite/gas/i386/i386.exp
index 34d0a2fab8ce2186ce94615e0df0a06c207ab61c..329efc71b0141a34f894cacbbdf7a7f11c4d994b 100644 (file)
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -638,6 +638,7 @@ if [gas_32_check] then {
     run_dump_test "nop-1-suffix"
     run_list_test "nop-1" "-aln"
     run_dump_test "nop-2"
+    run_list_test "nop-7" "-aln"
     run_dump_test "optimize-1"
     run_dump_test "optimize-1a"
     run_dump_test "optimize-2"

diff --git a/gas/testsuite/gas/i386/nop-7.l b/gas/testsuite/gas/i386/nop-7.l
new file mode 100644 (file)
index 0000000..2a74137
--- /dev/null
+++ b/gas/testsuite/gas/i386/nop-7.l
@@ -0,0 +1,6 @@
+[      ]*[0-9]+[       ]+\.text
+[      ]*[0-9]+[       ]+nops:
+[      ]*[0-9]+[       ]+.... F88D7426[        ]+clc; \.nops 4; stc
+[      ]*[0-9]+[       ]+00F9
+[      ]*[0-9]+[       ]+.... C3[      ]+ret
+#pass

diff --git a/gas/testsuite/gas/i386/nop-7.s b/gas/testsuite/gas/i386/nop-7.s
new file mode 100644 (file)
index 0000000..8018f21
--- /dev/null
+++ b/gas/testsuite/gas/i386/nop-7.s
@@ -0,0 +1,6 @@
+       .text
+nops:
+       clc; .nops 4; stc
+       ret
+
+       .end