This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [Patch][gas][arm] Fix testcase on arm-*pe* targets
- From: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>
- To: Matthew Malcomson <Matthew dot Malcomson at arm dot com>, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Cc: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, "nickc at redhat dot com" <nickc at redhat dot com>, nd <nd at arm dot com>
- Date: Tue, 6 Nov 2018 17:11:33 +0000
- Subject: Re: [Patch][gas][arm] Fix testcase on arm-*pe* targets
- References: <VI1PR0801MB2014373EEFE28BAE263E46F4E0CB0@VI1PR0801MB2014.eurprd08.prod.outlook.com>
On 06/11/2018 16:52, Matthew Malcomson wrote:
> The PE target can insert NOP's for padding to 4 byte alignment.
> This was causing a testcase failure, this commit fixes the testcase.
>
> This commit also escapes some full-stops in the testcase regexp.
>
> Ok for trunk?
> Regtested with cross-build for target arm-none-elf and arm-wince-pe.
>
> n.b. I don't have commit rights.
>
> gas/ChangeLog:
>
> 2018-11-06 Matthew Malcomson <matthew.malcomson@arm.com>
>
> * testsuite/gas/arm/neon-cond-bad_t2.d: Fix testcase for PE
> target.
>
Pushed.
R.
>
>
> ############### Attachment also inlined for ease of reply ###############
>
>
> diff --git a/gas/testsuite/gas/arm/neon-cond-bad_t2.d b/gas/testsuite/gas/arm/neon-cond-bad_t2.d
> index abff08431e1618718b8885da4e8465dc0732b459..47717ba4775ded7fc67e4aae269a801fd2888d81 100644
> --- a/gas/testsuite/gas/arm/neon-cond-bad_t2.d
> +++ b/gas/testsuite/gas/arm/neon-cond-bad_t2.d
> @@ -31,8 +31,8 @@ Disassembly of section \.text:
> 0[0-9a-f]+ <[^>]+> ef21 0d02 vsubeq\.f32 d0, d1, d2
> 0[0-9a-f]+ <[^>]+> ef22 0d44 vsubeq\.f32 q0, q1, q2
> 0[0-9a-f]+ <[^>]+> bf04 itt eq
> -0[0-9a-f]+ <[^>]+> ffb6 1602 vcvteq.f16.f32 d1, q1
> -0[0-9a-f]+ <[^>]+> ffb6 2701 vcvteq.f32.f16 q1, d1
> +0[0-9a-f]+ <[^>]+> ffb6 1602 vcvteq\.f16\.f32 d1, q1
> +0[0-9a-f]+ <[^>]+> ffb6 2701 vcvteq\.f32\.f16 q1, d1
> 0[0-9a-f]+ <[^>]+> bf04 itt eq
> 0[0-9a-f]+ <[^>]+> ffb9 0701 vabseq\.f32 d0, d1
> 0[0-9a-f]+ <[^>]+> ffb9 0742 vabseq\.f32 q0, q1
> @@ -56,3 +56,5 @@ Disassembly of section \.text:
> 0[0-9a-f]+ <[^>]+> eea0 1b10 vdupeq\.32 q0, r1
> 0[0-9a-f]+ <[^>]+> ffb4 0c01 vdupeq\.32 d0, d1\[0\]
> 0[0-9a-f]+ <[^>]+> ffbc 0c41 vdupeq\.32 q0, d1\[1\]
> +# PE targets can insert an extra NOP for padding -- avoid that breaking tests.
> +#pass
>
>
> testcase-fix.patch
>
> diff --git a/gas/testsuite/gas/arm/neon-cond-bad_t2.d b/gas/testsuite/gas/arm/neon-cond-bad_t2.d
> index abff08431e1618718b8885da4e8465dc0732b459..47717ba4775ded7fc67e4aae269a801fd2888d81 100644
> --- a/gas/testsuite/gas/arm/neon-cond-bad_t2.d
> +++ b/gas/testsuite/gas/arm/neon-cond-bad_t2.d
> @@ -31,8 +31,8 @@ Disassembly of section \.text:
> 0[0-9a-f]+ <[^>]+> ef21 0d02 vsubeq\.f32 d0, d1, d2
> 0[0-9a-f]+ <[^>]+> ef22 0d44 vsubeq\.f32 q0, q1, q2
> 0[0-9a-f]+ <[^>]+> bf04 itt eq
> -0[0-9a-f]+ <[^>]+> ffb6 1602 vcvteq.f16.f32 d1, q1
> -0[0-9a-f]+ <[^>]+> ffb6 2701 vcvteq.f32.f16 q1, d1
> +0[0-9a-f]+ <[^>]+> ffb6 1602 vcvteq\.f16\.f32 d1, q1
> +0[0-9a-f]+ <[^>]+> ffb6 2701 vcvteq\.f32\.f16 q1, d1
> 0[0-9a-f]+ <[^>]+> bf04 itt eq
> 0[0-9a-f]+ <[^>]+> ffb9 0701 vabseq\.f32 d0, d1
> 0[0-9a-f]+ <[^>]+> ffb9 0742 vabseq\.f32 q0, q1
> @@ -56,3 +56,5 @@ Disassembly of section \.text:
> 0[0-9a-f]+ <[^>]+> eea0 1b10 vdupeq\.32 q0, r1
> 0[0-9a-f]+ <[^>]+> ffb4 0c01 vdupeq\.32 d0, d1\[0\]
> 0[0-9a-f]+ <[^>]+> ffbc 0c41 vdupeq\.32 q0, d1\[1\]
> +# PE targets can insert an extra NOP for padding -- avoid that breaking tests.
> +#pass
>