From: Jan Beulich <jbeulich@suse.com>
To: Mark Harmstone <mark@harmstone.com>
Cc: binutils@sourceware.org, wej22007@outlook.com, zac.walker@linaro.org
Subject: Re: [PATCH 2/5] Skip ELF-specific tests when targeting pe-aarch64
Date: Thu, 22 Dec 2022 08:38:06 +0100 [thread overview]
Message-ID: <f83a1456-1347-2803-3cf7-ee7d46803d20@suse.com> (raw)
In-Reply-To: <ba3342e7-0d9d-e004-c5cc-ff8eb98de541@harmstone.com>
On 21.12.2022 21:19, Mark Harmstone wrote:
> On 16/12/22 07:07, Jan Beulich wrote:
>> On 16.12.2022 03:13, Mark Harmstone wrote:
>>> There's currently an awful lot of test failures for pe-aarch64, because
>>> of tests that assume that aarch64 implies ELF. This sets notarget for
>>> these.
>> Hmm, not very nice, but perhaps okay as a first step. However, ...
>>
>>> ---
>>> binutils/testsuite/binutils-all/objcopy.exp | 1 +
>>> gas/testsuite/gas/aarch64/adr_1.d | 1 +
>>> gas/testsuite/gas/aarch64/advsimd-mov-bad.d | 1 +
>>> gas/testsuite/gas/aarch64/b_1.d | 1 +
>>> gas/testsuite/gas/aarch64/beq_1.d | 1 +
>>> gas/testsuite/gas/aarch64/bfloat16-directive-be.d | 1 +
>>> gas/testsuite/gas/aarch64/bfloat16-directive-le.d | 1 +
>>> gas/testsuite/gas/aarch64/codealign.d | 2 +-
>>> gas/testsuite/gas/aarch64/codealign_1.d | 1 +
>>> gas/testsuite/gas/aarch64/dwarf.d | 1 +
>>> gas/testsuite/gas/aarch64/ilp32-basic.d | 1 +
>>> gas/testsuite/gas/aarch64/int-insns.d | 1 +
>>> gas/testsuite/gas/aarch64/ldr_1.d | 1 +
>>> gas/testsuite/gas/aarch64/litpool.d | 2 +-
>>> gas/testsuite/gas/aarch64/mapmisc.d | 2 +-
>>> gas/testsuite/gas/aarch64/mapping.d | 2 +-
>>> gas/testsuite/gas/aarch64/mapping2.d | 2 +-
>>> gas/testsuite/gas/aarch64/mapping3.d | 2 +-
>>> gas/testsuite/gas/aarch64/mapping4.d | 2 +-
>>> gas/testsuite/gas/aarch64/mapping_5.d | 1 +
>>> gas/testsuite/gas/aarch64/mapping_6.d | 1 +
>>> gas/testsuite/gas/aarch64/mops_invalid_2.d | 1 +
>>> gas/testsuite/gas/aarch64/movw_label.d | 1 +
>>> gas/testsuite/gas/aarch64/optional.d | 1 +
>>> gas/testsuite/gas/aarch64/pac_ab_key.d | 1 +
>>> gas/testsuite/gas/aarch64/pac_negate_ra_state.d | 1 +
>>> gas/testsuite/gas/aarch64/pr20364.d | 1 +
>>> gas/testsuite/gas/aarch64/pr27217.d | 1 +
>>> gas/testsuite/gas/aarch64/pr29519.d | 1 +
>>> gas/testsuite/gas/aarch64/programmer-friendly.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-data.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g0-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g0.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g0_nc-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g0_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g1-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g1_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_g2.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_hi12-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_hi12.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12-ilp32-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12-ldst16.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12-ldst32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12-ldst64.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12-ldst8.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12_nc-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12_nc-ldst16.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12_nc-ldst32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12_nc-ldst64.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12_nc-ldst8.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-dtprel_lo12_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-gotoff_g0_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-gotoff_g1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-gottprel_g0_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-gottprel_g1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-insn.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g0.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g0_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g1_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g2.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g2_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-prel_g3.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsdesc_off_g0_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsdesc_off_g1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsgd_g0_nc.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsgd_g1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsldm-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsldm-ilp32-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsldm-page-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsldm-page-ilp32-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsldm_lo12_nc-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tlsldm_lo12_nc-ilp32-1.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst16-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst16.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst32-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst64-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst64.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst8-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12-ldst8.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst16-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst16.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst32-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst64-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst64.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst8-ilp32.d | 1 +
>>> gas/testsuite/gas/aarch64/reloc-tprel_lo12_nc-ldst8.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_1.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_10.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_11.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_12.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_13.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_14.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_15.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_16.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_17.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_18.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_19.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_2.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_20.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_21.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_22.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_23.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_24.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_25.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_26.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_27.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_28.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_3.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_4.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_5.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_6.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_7.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_8.d | 1 +
>>> gas/testsuite/gas/aarch64/sve-movprfx_9.d | 1 +
>>> gas/testsuite/gas/aarch64/symbol-variant_pcs-1.d | 1 +
>>> gas/testsuite/gas/aarch64/symbol-variant_pcs-2.d | 1 +
>>> gas/testsuite/gas/aarch64/symbol-variant_pcs-3.d | 1 +
>>> gas/testsuite/gas/aarch64/system.d | 1 +
>>> gas/testsuite/gas/aarch64/tail_padding.d | 1 +
>>> gas/testsuite/gas/aarch64/tbz_1.d | 1 +
>>> gas/testsuite/gas/aarch64/tls-desc.d | 1 +
>>> gas/testsuite/gas/aarch64/tls.d | 1 +
>>> gas/testsuite/gas/pe/big-obj.d | 1 +
>> ... this clearly isn't an ELF test and hence can't fall into here.
>> At least for PE tests I think it would be better if they really worked.
>> If anything this should be xfail-ed imo (in a separate patch with a
>> suitable description).
>
> Thanks Jan. This patch isn't essential, so I'll save it for a later date if it's contentious. The other four are the important ones.
FTAOD my comment was merely about pe/big-obj.d. So perhaps you want to
split that off and get the rest of this change in?
Jan
next prev parent reply other threads:[~2022-12-22 7:38 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-16 2:13 [PATCH 1/5] Fix size of external_reloc for pe-aarch64 Mark Harmstone
2022-12-16 2:13 ` [PATCH 2/5] Skip ELF-specific tests when targeting pe-aarch64 Mark Harmstone
2022-12-16 7:07 ` Jan Beulich
2022-12-21 20:19 ` Mark Harmstone
2022-12-22 7:38 ` Jan Beulich [this message]
2022-12-16 2:13 ` [PATCH 3/5] Add pe-aarch64 relocations Mark Harmstone
2022-12-16 2:13 ` [PATCH 4/5] Add .secrel32 for pe-aarch64 Mark Harmstone
2022-12-16 2:14 ` [PATCH 5/5] Add aarch64-w64-mingw32 target Mark Harmstone
2022-12-16 7:28 ` Jan Beulich
2022-12-18 22:44 ` Martin Storsjö
2022-12-20 13:18 ` Mark Harmstone
2022-12-16 7:03 ` [PATCH 1/5] Fix size of external_reloc for pe-aarch64 Jan Beulich
2022-12-16 10:47 ` Tamar Christina
2022-12-20 12:59 ` Mark Harmstone
2022-12-20 13:10 ` Jan Beulich
2022-12-20 13:38 ` Tamar Christina
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f83a1456-1347-2803-3cf7-ee7d46803d20@suse.com \
--to=jbeulich@suse.com \
--cc=binutils@sourceware.org \
--cc=mark@harmstone.com \
--cc=wej22007@outlook.com \
--cc=zac.walker@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).