From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-x336.google.com (mail-wm1-x336.google.com [IPv6:2a00:1450:4864:20::336]) by sourceware.org (Postfix) with ESMTPS id 84BF23858D1E for ; Wed, 21 Dec 2022 20:19:11 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 84BF23858D1E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=harmstone.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-wm1-x336.google.com with SMTP id o15so11958461wmr.4 for ; Wed, 21 Dec 2022 12:19:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :sender:from:to:cc:subject:date:message-id:reply-to; bh=p7KqAW7geN0+cLdmDNvFnRhbOWRRZkq77Nq7bu9L4F0=; b=fSKjnRLNmxzUzAE73K4yorxWIa7TFc+G2ERp5hLY7b//hsHU446mMPPcIkWweePLj7 4+mN+Stwu35wIwlPCFnwoisFiQBt9Kdsqu4N/iw0nX0L0Z3kc12A1RfOQYXzlGd8gjAH BgdH2zvREPRodg6ZqAeJavY9TqPHf/xcuw6Ed84Ar8J+QPaJgodRp2a2IkGSvxTfcdj3 O+KvCkn5VAa8eliivguPMIwsO7hSkXpBLlkMEEHDWzA+voYekPfmULjIlQC0Chro8EXu +Honm998WSQkEJmr3Sv6L88YRinqkMRjLbqgP9z7GOlMMNDzDIKB6epzx3KJU5a2CqaA uSRA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :sender:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=p7KqAW7geN0+cLdmDNvFnRhbOWRRZkq77Nq7bu9L4F0=; b=dpOPgmQiEocT6mg5HY4ebpdVmYRqVAr0V4MzZ4yMkk7oBGM/LDpiJkuixUWjlWgdOf 5NsHHITRszzQigJT9nQc3cV3ImesnSKa/9pfMwe1yNXkJQxViPcjF3iPUEZJk2F120NK n6cvLR42X5EuXUitMwQiMJ0PqItph6OTKgYD2uPdfYYZGZrHAbUM//l80c02uxEOsFkT XDD3p+nJhNndxaYDsHMWvpnvgqO+XmlbifXT89w3fih17C7ZYW7THEhL4N74oEEqB1vx KEABxxYzZeKHx8c+vbCHEFY89gDWvTHjf1tMZnhBU5RZp0LDgsVSM37pE9kmpGzdrkx/ NtvA== X-Gm-Message-State: AFqh2kpVc1d2cY1dVxRvY7wCHNMB6KYUGtuVnELqOcmY42gnMMbES398 ZmdGQg9aXGMdgfMW4RlI1bE= X-Google-Smtp-Source: AMrXdXsFuxbkHoYmQZh+Z+CxxJBiNwHThQTeC0gPJPf64CwnlQ6RfSQsqXzlI5WVqczvXmfupjgtRQ== X-Received: by 2002:a05:600c:4e51:b0:3d1:e1f4:21d1 with SMTP id e17-20020a05600c4e5100b003d1e1f421d1mr2832190wmq.26.1671653950093; Wed, 21 Dec 2022 12:19:10 -0800 (PST) Received: from ?IPV6:2a02:8010:64ea:0:8eb8:7eff:fe53:9d5f? ([2a02:8010:64ea:0:8eb8:7eff:fe53:9d5f]) by smtp.googlemail.com with ESMTPSA id 5-20020a05600c228500b003d1cc0464a2sm3322791wmf.8.2022.12.21.12.19.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 21 Dec 2022 12:19:09 -0800 (PST) Sender: Mark Harmstone Message-ID: Date: Wed, 21 Dec 2022 20:19:07 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH 2/5] Skip ELF-specific tests when targeting pe-aarch64 To: Jan Beulich Cc: binutils@sourceware.org, wej22007@outlook.com, zac.walker@linaro.org References: <20221216021400.22309-1-mark@harmstone.com> <20221216021400.22309-2-mark@harmstone.com> <39314963-c931-4013-7821-2012cd8871f7@suse.com> Content-Language: en-US From: Mark Harmstone In-Reply-To: <39314963-c931-4013-7821-2012cd8871f7@suse.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-5.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,KAM_NUMSUBJECT,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: 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). > > Jan 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. Mark