[PATCH, BINUTILS, ARM, 2/3] Add instruction SB for AArch32
Sudakshina Das
Sudi.Das@arm.com
Thu Oct 4 13:35:00 GMT 2018
Hi
On 01/10/18 18:31, Sudakshina Das wrote:
> Hi
>
> On 01/10/18 15:57, Hans-Peter Nilsson wrote:
>> On Fri, 28 Sep 2018, Sudakshina Das wrote:
>>>>> --- /dev/null
>>>>> +++ b/gas/testsuite/gas/arm/sb-thumb1.d
>>>>> @@ -0,0 +1,13 @@
>>>>> +#objdump: -dr --prefix-addresses --show-raw-insn
>>>>> +#name: SB instruction (Thumb)
>>>>> +#source: sb.s
>>>>> +#as: -march=armv8.5-a -mthumb
>>>>> +# This test is only valid on ELF based ports.
>>>>> +#not-target: *-*-*coff *-*-pe *-*-wince *-*-*aout* *-*-netbsd
>>
>> A gotcha here: the gas run-dump-test "not-target" is now
>> "notarget" due to recent framework consolidation (with
>> "notarget" from the ld test-suite, which had precedence).
>>
>> brgds, H-P
>>
>
> Ah yes, I also noticed the error-output to error_output change.
> Thanks for bringing it to my notice. Will merge in and rebase
> the patches.
>
Please find the new patch attached:
*** include/ChangeLog ***
2018-xx-xx Sudakshina Das <sudi.das@arm.com>
* opcode/arm.h (ARM_EXT2_SB): New.
(ARM_ARCH_V8_5A): Add ARM_EXT2_SB by default.
*** opcodes/ChangeLog ***
2018-xx-xx Sudakshina Das <sudi.das@arm.com>
* arm-dis.c (arm_opcodes): Add sb.
(thumb32_opcodes): Likewise.
*** gas/ChangeLog ***
2018-xx-xx Sudakshina Das <sudi.das@arm.com>
* config/tc-arm.c (arm_ext_sb): New.
(insns): Add new sb instruction.
(arm_extensions): Add "sb".
* doc/c-arm.texi: Document the above.
* testsuite/gas/arm/sb-bad.d: New test.
* testsuite/gas/arm/sb-bad.l: New test.
* testsuite/gas/arm/sb-thumb1.d: New test.
* testsuite/gas/arm/sb-thumb2.d: New test.
* testsuite/gas/arm/sb.s: New test.
* testsuite/gas/arm/sb1.d: New test.
* testsuite/gas/arm/sb2.d: New test.
Thanks
Sudi
> Thanks
> Sudi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rb9974.patch
Type: text/x-patch
Size: 7625 bytes
Desc: rb9974.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20181004/d709df12/attachment.bin>
More information about the Binutils
mailing list