Message ID | or1q6xoi2w.fsf@lxoliva.fsfla.org |
---|---|
State | New |
Headers |
Return-Path: <gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id DF3CE384AB4F for <patchwork@sourceware.org>; Mon, 22 Apr 2024 09:37:21 +0000 (GMT) X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from mail-pg1-x536.google.com (mail-pg1-x536.google.com [IPv6:2607:f8b0:4864:20::536]) by sourceware.org (Postfix) with ESMTPS id EB28B3858C98 for <gcc-patches@gcc.gnu.org>; Mon, 22 Apr 2024 09:36:05 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org EB28B3858C98 Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org EB28B3858C98 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::536 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1713778571; cv=none; b=p7nJ/mJsjE3HJ7WZA3N+5AgjDejwXFbj4QELr4jgReTPBs33YfcgO/rb+0mY9xPZ3Z1tqGuzdf2Rh56TUfBhmyiVGLZmZGEDfh9B9Eb1neFx2XM0M/sby3sEI/LF+NM13LhKBLlnS+caOWnPHw0l7ZytVmsb7qTfeZW/Ax4unfE= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1713778571; c=relaxed/simple; bh=+b7kdsZPQCdj5VnVntTZNjg9w4le5OI4P8KJSHjV9+0=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=ww97Q61yg2oM8svoYA7exkw3hhM3I2vRUZxPbN8pLoHebYfrMeEUvUjso90PEJjdEhNgUjWSPG1I5hEOFWfa9tHnNTljJR4xwpfsy6zh1M5QZ1g/jzzyBeoAu88DhUwsOc5ztTQ0Ik+eP7qL1QljKR2/OMzfLBrElVYTnSDehGc= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-pg1-x536.google.com with SMTP id 41be03b00d2f7-53fa455cd94so3168418a12.2 for <gcc-patches@gcc.gnu.org>; Mon, 22 Apr 2024 02:36:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1713778565; x=1714383365; darn=gcc.gnu.org; h=mime-version:user-agent:message-id:in-reply-to:date:references :organization:subject:cc:to:from:from:to:cc:subject:date:message-id :reply-to; bh=TBNvWZhi/RyylsxbIx5OIn0IwM7CFKwIGyYMWEapoRQ=; b=AOWJWSbAMo2GKV/ajbEPU0Sj8wnv0pU8rU3eYSwXBzCXfUke8OfTXwvXIxf9EBv0p9 TkyXaZcclY3R0QomggvH04A/3oPdZj7pOcX3T5st1/piZe2+u6myUp2JQVMd5BpsJ5hS W/Te7pgBZhMrG0jwEVjUdtvygE2uxCx8eLDYy7wMjK84sEiTg+v2GDWXznmRCXc201Me WH7mSe2Cv4Iq4esLXzVGASucHCBEpLIueX0aFpRSrxMg6aS3hq/QGmLKYOYmqQPUCCIZ e3HimM6cjFWidOuYe55Sk9E9+8jRyjxtpOh9ygdvsWBFsTDupX8rrT82LDkRxXdeTQUZ 22Ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713778565; x=1714383365; h=mime-version:user-agent:message-id:in-reply-to:date:references :organization:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=TBNvWZhi/RyylsxbIx5OIn0IwM7CFKwIGyYMWEapoRQ=; b=qIL1qaYXlPXZpNTnhtR9BPM32/l1oj3k/YyKcG/4wFztn0bEDaVz2eO97fYcoOGcMz zo5lnCn62zf2kbhs7LNH8qxmLtAgmvl9AJ7j02FclSJY/tDql9u48zR3a4QXRJc8V2+F k4Rq062nEbg3ClCu5qqHldFJrTUXEVSYTkTyRCjSIixG5xrJ+BOA2VefqJJOiLzliPgq Q5Pmrh2Ap3L3UbaP7GY0FYWyQ3p8Uda3GAoUGsEZUvIBYN3V3Rm1tObAsnf7BAH+Dm/O cz05hKhNIGJHi7mGgdv2c/I7oBYH5oyAXmfntvk5/VF43Y4EbTHnULhMWG4EajmVQqMm zGaQ== X-Gm-Message-State: AOJu0YwOz2LTnhRPtJDEbb1plQhahvq7gVFJV620eghjIqOdVS1UzrJW VDhafJrEPlCDb7yolZvptX96KqFZHjh0mo2Vs2ICd7sKTv4Y6fW4BIwMq90RaJCZt6pV5JXpQ8m 8UQ== X-Google-Smtp-Source: AGHT+IEllf4YdCnWbRHFxF8q81JvopGCWNMlFBqg0v+8N5qDNw+YiIB2XPklDDxJL+GYTH5bylheNg== X-Received: by 2002:a17:90a:5213:b0:2a7:600e:ce0 with SMTP id v19-20020a17090a521300b002a7600e0ce0mr7137058pjh.42.1713778564505; Mon, 22 Apr 2024 02:36:04 -0700 (PDT) Received: from free.home ([2804:7f1:218b:5e2:c43b:e0c4:71a:241b]) by smtp.gmail.com with ESMTPSA id s12-20020a17090a5d0c00b002a005778f51sm1527658pji.50.2024.04.22.02.36.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 22 Apr 2024 02:36:03 -0700 (PDT) Received: from livre (livre.home [172.31.160.2]) by free.home (8.15.2/8.15.2) with ESMTPS id 43M9ZpHK019850 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NOT); Mon, 22 Apr 2024 06:35:51 -0300 From: Alexandre Oliva <oliva@adacore.com> To: gcc-patches@gcc.gnu.org Cc: Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>, Mike Stump <mikestump@comcast.net>, David Edelsohn <dje.gcc@gmail.com>, Segher Boessenkool <segher@kernel.crashing.org>, Kewen Lin <linkw@gcc.gnu.org> Subject: [PATCH] ppc: testsuite: vec-mul requires vsx runtime Organization: Free thinker, does not speak for AdaCore References: <orwnf3fa5a.fsf@lxoliva.fsfla.org> Date: Mon, 22 Apr 2024 06:35:51 -0300 In-Reply-To: <orwnf3fa5a.fsf@lxoliva.fsfla.org> (Alexandre Oliva's message of "Tue, 03 May 2022 02:38:41 -0300") Message-ID: <or1q6xoi2w.fsf@lxoliva.fsfla.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Scanned-By: MIMEDefang 2.84 X-Spam-Status: No, score=-12.1 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, WEIRD_QUOTING autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gcc-patches mailing list <gcc-patches.gcc.gnu.org> List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe> List-Archive: <https://gcc.gnu.org/pipermail/gcc-patches/> List-Post: <mailto:gcc-patches@gcc.gnu.org> List-Help: <mailto:gcc-patches-request@gcc.gnu.org?subject=help> List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe> Errors-To: gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org |
Series |
ppc: testsuite: vec-mul requires vsx runtime
|
|
Checks
Context | Check | Description |
---|---|---|
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 | success | Testing passed |
linaro-tcwg-bot/tcwg_gcc_check--master-aarch64 | success | Testing passed |
linaro-tcwg-bot/tcwg_gcc_build--master-arm | success | Testing passed |
linaro-tcwg-bot/tcwg_gcc_check--master-arm | success | Testing passed |
Commit Message
Alexandre Oliva
April 22, 2024, 9:35 a.m. UTC
Ping? https://gcc.gnu.org/pipermail/gcc-patches/2022-May/593947.html vec-mul is an execution test, but it only requires a powerpc_vsx_ok effective target, which is enough only for compile tests. In order to To check for runtime and execution environment support, we need to require vsx_hw. Make that a condition for execution, but still perform a compile test if the condition is not satisfied. Regstrapped on x86_64-linux-gnu and ppc64el-linux-gnu. Also tested with gcc-13 on ppc64-vx7r2 and ppc-vx7r2. Ok to install? for gcc/testsuite/ChangeLog * gcc.target/powerpc/vec-mul.c: Run on target vsx_hw, just compile otherwise. --- gcc/testsuite/gcc.target/powerpc/vec-mul.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
Comments
on 2024/4/22 17:35, Alexandre Oliva wrote: > Ping? > https://gcc.gnu.org/pipermail/gcc-patches/2022-May/593947.html > > > vec-mul is an execution test, but it only requires a powerpc_vsx_ok > effective target, which is enough only for compile tests. In order to > To check for runtime and execution environment support, we need to > require vsx_hw. Make that a condition for execution, but still > perform a compile test if the condition is not satisfied. > > Regstrapped on x86_64-linux-gnu and ppc64el-linux-gnu. Also tested with > gcc-13 on ppc64-vx7r2 and ppc-vx7r2. Ok to install? > > > for gcc/testsuite/ChangeLog > > * gcc.target/powerpc/vec-mul.c: Run on target vsx_hw, just > compile otherwise. > --- > gcc/testsuite/gcc.target/powerpc/vec-mul.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/gcc/testsuite/gcc.target/powerpc/vec-mul.c b/gcc/testsuite/gcc.target/powerpc/vec-mul.c > index bfcaf80719d1d..11da86159723f 100644 > --- a/gcc/testsuite/gcc.target/powerpc/vec-mul.c > +++ b/gcc/testsuite/gcc.target/powerpc/vec-mul.c > @@ -1,4 +1,5 @@ > -/* { dg-do run } */ > +/* { dg-do compile { target { ! vsx_hw } } } */ > +/* { dg-do run { target vsx_hw } } */ > /* { dg-require-effective-target powerpc_vsx_ok } */ Nit: It's useless to check powerpc_vsx_ok for vsx_hw, so powerpc_vsx_ok check can be moved to be with ! vsx_hw. OK with this nit tweaked, thanks! BR, Kewen > /* { dg-options "-mvsx -O3" } */ > > >
On Apr 23, 2024, "Kewen.Lin" <linkw@linux.ibm.com> wrote: >> -/* { dg-do run } */ >> +/* { dg-do compile { target { ! vsx_hw } } } */ >> +/* { dg-do run { target vsx_hw } } */ >> /* { dg-require-effective-target powerpc_vsx_ok } */ > Nit: It's useless to check powerpc_vsx_ok for vsx_hw, so powerpc_vsx_ok check > can be moved to be with ! vsx_hw. > OK with this nit tweaked, thanks! Thanks, here's what I'm pushing momentarily... ppc: testsuite: vec-mul requires vsx runtime vec-mul is an execution test, but it only requires a powerpc_vsx_ok effective target, which is enough only for compile tests. In order to check for runtime and execution environment support, we need to require vsx_hw. Make that a condition for execution, but still perform a compile test if the condition is not satisfied. for gcc/testsuite/ChangeLog * gcc.target/powerpc/vec-mul.c: Run on target vsx_hw, just compile otherwise. --- gcc/testsuite/gcc.target/powerpc/vec-mul.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/testsuite/gcc.target/powerpc/vec-mul.c b/gcc/testsuite/gcc.target/powerpc/vec-mul.c index bfcaf80719d1d..aa0ef7aa45acc 100644 --- a/gcc/testsuite/gcc.target/powerpc/vec-mul.c +++ b/gcc/testsuite/gcc.target/powerpc/vec-mul.c @@ -1,5 +1,5 @@ -/* { dg-do run } */ -/* { dg-require-effective-target powerpc_vsx_ok } */ +/* { dg-do compile { target { { ! vsx_hw } && powerpc_vsx_ok } } } */ +/* { dg-do run { target vsx_hw } } */ /* { dg-options "-mvsx -O3" } */ /* Test that the vec_mul builtin works as expected. */
diff --git a/gcc/testsuite/gcc.target/powerpc/vec-mul.c b/gcc/testsuite/gcc.target/powerpc/vec-mul.c index bfcaf80719d1d..11da86159723f 100644 --- a/gcc/testsuite/gcc.target/powerpc/vec-mul.c +++ b/gcc/testsuite/gcc.target/powerpc/vec-mul.c @@ -1,4 +1,5 @@ -/* { dg-do run } */ +/* { dg-do compile { target { ! vsx_hw } } } */ +/* { dg-do run { target vsx_hw } } */ /* { dg-require-effective-target powerpc_vsx_ok } */ /* { dg-options "-mvsx -O3" } */