From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19457 invoked by alias); 8 Jun 2011 10:40:00 -0000 Received: (qmail 19448 invoked by uid 22791); 8 Jun 2011 10:40:00 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from service87.mimecast.com (HELO service87.mimecast.com) (94.185.240.25) by sourceware.org (qpsmtpd/0.43rc1) with SMTP; Wed, 08 Jun 2011 10:39:46 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Wed, 08 Jun 2011 11:39:42 +0100 Received: from [10.1.67.34] ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.0); Wed, 8 Jun 2011 11:40:11 +0100 Message-ID: <4DEF516C.7060703@arm.com> Date: Wed, 08 Jun 2011 11:47:00 -0000 From: Richard Earnshaw User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-GB; rv:1.9.1.5) Gecko/20091204 Thunderbird/3.0 MIME-Version: 1.0 To: Janis Johnson CC: Mike Stump , "Joseph S. Myers" , "gcc-patches@gcc.gnu.org" Subject: Re: [testsuite] skip ARM tests with conflicting options References: <4DEE8FE7.50001@codesourcery.com> <4DEEB332.2070706@codesourcery.com> <4DEEDAF0.6090100@codesourcery.com> In-Reply-To: <4DEEDAF0.6090100@codesourcery.com> X-MC-Unique: 111060811394204101 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-06/txt/msg00638.txt.bz2 On 08/06/11 03:14, Janis Johnson wrote: > On 06/07/2011 06:25 PM, Mike Stump wrote: >> On Jun 7, 2011, at 4:24 PM, Janis Johnson wrote: >>> On 06/07/2011 02:07 PM, Joseph S. Myers wrote: >>>> On Tue, 7 Jun 2011, Janis Johnson wrote: >>>> >>>>> Several tests in gcc.target/arm use dg-options with -mcpu=3Dxxxx, whi= ch >>>>> causes compiler warnings or errors when the multilib flags include >>>>> -march=3Dyyyy. This patch causes those tests to be skipped. It also >>>>> prevents gcc.target/arm/20090811-1.c from running with multilibs that >>>>> would override -mcpu or -mfloat-abi options specified for the test. >>>> >>>> I think you should allow compatible -march options - for example, if=20 >>>> dg-options has -mcpu=3Dcortex-a8, allow -march=3Darmv7-a but disallow = all=20 >>>> other -march options. >>>> >>> Is this one OK? >> >> Not sure if the arm people want to review this or would rather I review = it... >> >> Let's give the arm folks a couple days to comment, if no objections, Ok. >> >> A point of warning, eventually, you'll discover that when a compiler def= aults to the argument you want to skip, that you'll needs slightly more pow= er to skip them. darwin ran into this with things like -m64, and eventuall= y had to do something like lp64. configure options like --with-cpu=3Darm9 = are the sort that can change the default. >=20 > Yes, I hope to hear from ARM people. >=20 > On ARM, the default from --with-cpu=3D is overridden by -march at > compile so it's not a problem for this particular set of tests. > As I said originally, this set is the tip of the iceberg and they > get more difficult. >=20 > Janis >=20 >=20 I'm worried by this whole approach of command-line checking. It works, just about, for testsuite variations set with target_list, but it won't work with options used to configure the compiler (eg --with-mode=3Dthumb, or --with-cpu=3D...). Perhaps a better approach would be a new dg- test that built a trivial file with all the options and disabled the test if that test failed for any reason. Something like: dg-target-compatible (target, , additional-opts) The test is only performed if target matches the current target. I'm not sure if this is something that can be easily cached (well, it might be possible if you could index off additional-opts and the default opts), so it might be that this test has to be re-run every time there is a test that needs it. R.