From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by sourceware.org (Postfix) with ESMTP id 4A8343858C35 for ; Fri, 9 Feb 2024 14:57:03 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 4A8343858C35 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=arm.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 4A8343858C35 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1707490625; cv=none; b=itZ1ypeFAunoisK/PZpAwQA+VzdxavzJ+eoH+Yxugdni1KLGtZt3lUklRWfvQGeQ6EovkFwSLQju3CaELdeJw4Ul21FAZZA1B/GVcC/tRgI0Voj0iB4OCvASkAXN92e5I8AoXJNhWOWlf0R7Zw2l/VEI04SzEUEb1aVW6wEeorM= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1707490625; c=relaxed/simple; bh=pG5lXbPdi/ORiqJ6J/n96fmhPxwt1sWB6b9WmoPNrtI=; h=Message-ID:Date:MIME-Version:Subject:To:From; b=JBRFVU289U47tlwarosYRgU6lG45ZEljHF541TRVVdHRXfz2by53fkJZfvBldQ8rvATqqJfScAwchfBD9C/V/tFHl4bG9O/33hT1HXprIMjOWr4KZT+XZfghI9YHKAld2RmE4VK//hzleiVaLAxP1kqlySbyqBrGZ5auG41+XlU= ARC-Authentication-Results: i=1; server2.sourceware.org Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 198B0DA7; Fri, 9 Feb 2024 06:57:44 -0800 (PST) Received: from [10.2.78.54] (e120077-lin.cambridge.arm.com [10.2.78.54]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 5D5993F762; Fri, 9 Feb 2024 06:57:01 -0800 (PST) Message-ID: Date: Fri, 9 Feb 2024 14:57:00 +0000 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] testsuite, arm: Fix testcase arm/pr112337.c to check for the options first Content-Language: en-GB To: Saurabh Jha , Richard Sandiford via Gcc-patches , Richard Sandiford , Kyrylo Tkachov References: From: "Richard Earnshaw (lists)" In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-3491.8 required=5.0 tests=BAYES_00,KAM_DMARC_NONE,KAM_DMARC_STATUS,KAM_LAZY_DOMAIN_SECURITY,KAM_SHORT,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE autolearn=no 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 30/01/2024 17:07, Saurabh Jha wrote: > Hey, > > Previously, this test was added to fix this bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112337. However, it did not check the compilation options before using them, leading to errors. > > This patch fixes the test by first checking whether it can use the options before using them. > > Tested for arm-none-eabi and found no regressions. The output of check-gcc with RUNTESTFLAGS="arm.exp=*" changed like this: > > Before: > # of expected passes 5963 > # of unexpected failures 64 > > After: > # of expected passes 5964 > # of unexpected failures 63 > > Ok for master? > > Regards, > Saurabh > > gcc/testsuite/ChangeLog: > > * gcc.target/arm/pr112337.c: Check whether we can use the compilation options before using them. My apologies for missing this earlier. It didn't show up in patchwork. That's most likely because the attachment is a binary blob instead of text/plain. That also means that the Linaro CI system hasn't seen this patch either. Please can you fix your mailer to add plain text patch files. -/* { dg-options "-O2 -march=armv8.1-m.main+fp.dp+mve.fp -mfloat-abi=hard" } */ +/* { dg-require-effective-target arm_hard_ok } */ +/* { dg-require-effective-target arm_v8_1m_mve_ok } */ +/* { dg-options "-O2 -mfloat-abi=hard" } */ +/* { dg-add-options arm_v8_1m_mve } */ This is moving in the right direction, but it adds more than necessary now: checking for, and adding -mfloat-abi=hard is not necessary any more as arm_v8_1m_mve_ok will work out what float-abi flags are needed to make the options work. (What's more, it will prevent the test from running if the base configuration of the compiler is incompatible with the hard float ABI, which is more than we need.). So please can you re-spin removing the hard-float check and removing that from dg-options. Thanks, R.