This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86: Update assembler tests for non-ELF targets
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Binutils <binutils at sourceware dot org>
- Date: Fri, 10 Aug 2018 11:30:39 -0700
- Subject: Re: [PATCH] x86: Update assembler tests for non-ELF targets
- References: <20180805170917.9826-1-hjl.tools@gmail.com>
On Sun, Aug 5, 2018 at 10:09 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> evex-no-scale.s has ELF directive:
>
> .section .probe, "", @progbits
>
> and non-ELF targets may pad text sections.
>
> * testsuite/gas/i386/i386.exp: Run evex-no-scale-32 and
> evex-no-scale-64 only for ELF targets.
> * testsuite/gas/i386/prefix32.s: Append ".p2align 4,0".
> * testsuite/gas/i386/prefix64.s: Likewise.
> * testsuite/gas/i386/prefix32.l: Updated.
> * testsuite/gas/i386/prefix64.l: Likewise.
> ---
> gas/ChangeLog | 9 +++++++++
> gas/testsuite/gas/i386/i386.exp | 6 ++++--
> gas/testsuite/gas/i386/prefix32.l | 2 ++
> gas/testsuite/gas/i386/prefix32.s | 2 ++
> gas/testsuite/gas/i386/prefix64.l | 2 ++
> gas/testsuite/gas/i386/prefix64.s | 2 ++
> 6 files changed, 21 insertions(+), 2 deletions(-)
>
> diff --git a/gas/ChangeLog b/gas/ChangeLog
> index 94b496bce9..5959b11f15 100644
> --- a/gas/ChangeLog
> +++ b/gas/ChangeLog
> @@ -1,3 +1,12 @@
> +2018-08-05 H.J. Lu <hongjiu.lu@intel.com>
> +
> + * testsuite/gas/i386/i386.exp: Run evex-no-scale-32 and
> + evex-no-scale-64 only for ELF targets.
> + * testsuite/gas/i386/prefix32.s: Append ".p2align 4,0".
> + * testsuite/gas/i386/prefix64.s: Likewise.
> + * testsuite/gas/i386/prefix32.l: Updated.
> + * testsuite/gas/i386/prefix64.l: Likewise.
> +
> 2018-08-05 Alan Modra <amodra@gmail.com>
>
> * config/tc-ppc.c (ppc_elf_suffix): Support @notoc.
> diff --git a/gas/testsuite/gas/i386/i386.exp b/gas/testsuite/gas/i386/i386.exp
> index 3cf1aa33fc..6ea4deef34 100644
> --- a/gas/testsuite/gas/i386/i386.exp
> +++ b/gas/testsuite/gas/i386/i386.exp
> @@ -225,7 +225,6 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_32_check]]
> run_dump_test "evex-lig512-intel"
> run_dump_test "evex-wig1"
> run_dump_test "evex-wig1-intel"
> - run_dump_test "evex-no-scale-32"
> run_dump_test "sse2avx"
> run_list_test "inval-avx" "-al"
> run_list_test "inval-avx512f" "-al"
> @@ -528,6 +527,8 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_32_check]]
> run_dump_test "nop-5"
> run_dump_test "nop-6"
>
> + run_dump_test "evex-no-scale-32"
> +
> if { [gas_64_check] } then {
> run_dump_test "att-regs"
> run_dump_test "intel-regs"
> @@ -738,7 +739,6 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_64_check]] t
> run_dump_test "x86-64-evex-lig512-intel"
> run_dump_test "x86-64-evex-wig1"
> run_dump_test "x86-64-evex-wig1-intel"
> - run_dump_test "evex-no-scale-64"
> run_dump_test "x86-64-sse2avx"
> run_list_test "x86-64-inval-avx" "-al"
> run_list_test "x86-64-inval-avx512f" "-al"
> @@ -1023,6 +1023,8 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_64_check]] t
> run_dump_test "x86-64-nop-4"
> run_dump_test "x86-64-nop-5"
> run_dump_test "x86-64-nop-6"
> +
> + run_dump_test "evex-no-scale-64"
> }
>
> set ASFLAGS "$old_ASFLAGS"
I backported the 386.exp change to 2.31 branch.
--
H.J.