[PATCH][GAS][MSP430] Improve NOP warnings/insertion behaviour around interrupt state changes
Nick Clifton
nickc@redhat.com
Tue Nov 27 12:31:00 GMT 2018
Hi Jozef,
> If the patch is acceptable, I would appreciate if someone would apply it for
> me, as I don't have write access.
Approved and applied.
Note - there was one bug in the patch:
> diff --git a/gas/config/tc-msp430.c b/gas/config/tc-msp430.c
[...]
> @@ -2654,37 +2733,76 @@ msp430_operands (struct msp430_opcode_s * opcode, char * line)
[...]
>
> - if (gen_interrupt_nops)
> - /* Emit a NOP between interrupt enable/disable.
> - See 1.3.4.1 of the MSP430x5xx User Guide. */
> - doit = TRUE;
> - break;
> -
> case NOP_CHECK_CPU12:
This deleted the "break" statement at the end of NOP_CHECK_INTERRUPT case.
I assumed that your intent was not to fall through into the NOP_CHECK_CPU12
case, so I restored the break statement.
Cheers
Nick
More information about the Binutils
mailing list