From: Alan Modra Date: Thu, 8 Mar 2018 04:04:09 +0000 (+1030) Subject: Really remove unnecessary power9 group terminating nop X-Git-Tag: binutils-2_31~1101 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6a7524c6aa18f5afa4977314eb01f0360e752789;p=external%2Fbinutils.git Really remove unnecessary power9 group terminating nop Oops, not tested well enough. -mpower9 sets all the PPC_OPCODE_POWERn for n <= 9. * config/tc-ppc.c (ppc_handle_align): Correct last patch. Really don't emit a group terminating nop for power9. Simplify cpu tests. --- diff --git a/gas/ChangeLog b/gas/ChangeLog index 7670ee2..8a6161d 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,5 +1,11 @@ 2018-03-08 Alan Modra + * config/tc-ppc.c (ppc_handle_align): Correct last patch. Really + don't emit a group terminating nop for power9. Simplify cpu + tests. + +2018-03-08 Alan Modra + * config/tc-ppc.c (ppc_handle_align): Don't emit a group terminating nop for power9. diff --git a/gas/config/tc-ppc.c b/gas/config/tc-ppc.c index dc63d60..ff76221 100644 --- a/gas/config/tc-ppc.c +++ b/gas/config/tc-ppc.c @@ -6549,8 +6549,7 @@ ppc_handle_align (struct frag *fragP) md_number_to_chars (dest, 0x60000000, 4); if ((ppc_cpu & PPC_OPCODE_POWER6) != 0 - || (ppc_cpu & PPC_OPCODE_POWER7) != 0 - || (ppc_cpu & PPC_OPCODE_POWER8) != 0) + && (ppc_cpu & PPC_OPCODE_POWER9) == 0) { /* For power6, power7, and power8, we want the last nop to be a group terminating one. Do this by inserting an @@ -6570,8 +6569,7 @@ ppc_handle_align (struct frag *fragP) dest = group_nop->fr_literal; } - if ((ppc_cpu & PPC_OPCODE_POWER7) != 0 - || (ppc_cpu & PPC_OPCODE_POWER8) != 0) + if ((ppc_cpu & PPC_OPCODE_POWER7) != 0) { if (ppc_cpu & PPC_OPCODE_E500MC) /* e500mc group terminating nop: "ori 0,0,0". */