From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yb1-xb31.google.com (mail-yb1-xb31.google.com [IPv6:2607:f8b0:4864:20::b31]) by sourceware.org (Postfix) with ESMTPS id D7BDF385840F for ; Mon, 14 Aug 2023 02:46:58 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org D7BDF385840F Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-yb1-xb31.google.com with SMTP id 3f1490d57ef6-cfcebc33d04so3825825276.2 for ; Sun, 13 Aug 2023 19:46:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1691981218; x=1692586018; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=vL4YsS0uS0ATYBBfpD6VYxXAorWYhRPm2rYKsD4dIL4=; b=guc/XFXr7/OM17BM8+szuiHN3A1glLNJ7P8BT0iOQ8ndlHkfTblpABzaBHc0gDF+cH af18DMsufIqhgZ5UCsT7qRYRqJbsdjROjLf/y41mh6CKO1yJqKPuJJlQxcoUOKOeqLZw U+9A+M/88GHhlL6dWJCqOa06CMWXZA8Pe0si+ISs8UDpFdkqfucY1Y5dXTUViqhFLJzT PMddgEbr9YxKwKoF6ZQmgGsWsp4TeeWeyPTYw5HP+Jsnbm3wAThQR7nTYfl6UjZRucJG gw2uXlQeFuz//0qBYUW5HmLwuQdkWjUMMWXYLmz1SWfPlvo1hA1NkqYgIkb3B8RJc3ba xOdg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691981218; x=1692586018; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=vL4YsS0uS0ATYBBfpD6VYxXAorWYhRPm2rYKsD4dIL4=; b=Hfm+z0yNVPFG0xun8KDLS+pfFov8+2tA6IYPAXaJmfE41EaIfUTZ8uATufsJEwxWyk V+PkVop4gSy5+VQynm3rHOL26i1apDbFnUUzeEtG0p/33EEjeggbFHsLsVm3fCRLVk2e 69YZqSj0V4D14FhiHdJltFVgOZpa2bMKwa74Ti10FekXLqGVW4JuYYMKT/KZ55bRuvE5 KHNQtJROIStso4yoCkLRABZMzbny4sm6VlcxtvdVhPq3LEeircztKm4W8OW0LYefaYZD GNolyqm407gaT01eqbaEAdB4fxwonTRs2vgbTkmGa9UGNB0g+od9S8uzSJaHwzpXbPIG fRNw== X-Gm-Message-State: AOJu0YzrSMhwmi9ugOoX33u/JOKJ4Lha0lPXPuhzldo5xDtYr87w3Euc zhNT95UeVK6yCORRYqp1LDdGxVPqbhVTgLX/IdUkW2d6HlE= X-Google-Smtp-Source: AGHT+IG3CxMEF7Teik6lk/4Vi3kpGj+FpvnlUGidQyNtUHeTI5pZLZS5IuUrHxhzEQzGyzeDDHqHGgouPnn8akcJVY4= X-Received: by 2002:a25:ac10:0:b0:d63:839:989 with SMTP id w16-20020a25ac10000000b00d6308390989mr9388716ybi.0.1691981218186; Sun, 13 Aug 2023 19:46:58 -0700 (PDT) MIME-Version: 1.0 References: <20230814024600.1594913-1-hongtao.liu@intel.com> In-Reply-To: <20230814024600.1594913-1-hongtao.liu@intel.com> From: Hongtao Liu Date: Mon, 14 Aug 2023 10:54:17 +0800 Message-ID: Subject: Re: [PATCH] Generate vmovapd instead of vmovsd for moving DFmode between SSE_REGS. To: liuhongt Cc: gcc-patches@gcc.gnu.org, hjl.tools@gmail.com, Uros Bizjak Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,GIT_PATCH_0,KAM_SHORT,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: cc On Mon, Aug 14, 2023 at 10:46=E2=80=AFAM liuhongt w= rote: > > vmovapd can enable register renaming and have same code size as > vmovsd. Similar for vmovsh vs vmovaps, vmovaps is 1 byte less than > vmovsh. > > When TARGET_AVX512VL is not available, still generate > vmovsd/vmovss/vmovsh to avoid vmovapd/vmovaps zmm16-31. > > Bootstrapped and regtested on x86_64-pc-linux-gnu{-m32,}. > Ok for trunk? > > gcc/ChangeLog: > > * config/i386/i386.md (movdf_internal): Generate vmovapd instead = of > vmovsd when moving DFmode between SSE_REGS. > (movhi_internal): Generate vmovdqa instead of vmovsh when > moving HImode between SSE_REGS. > (mov_internal): Use vmovaps instead of vmovsh when > moving HF/BFmode between SSE_REGS. > > gcc/testsuite/ChangeLog: > > * gcc.target/i386/pr89229-4a.c: Adjust testcase. > --- > gcc/config/i386/i386.md | 20 +++++++++++++++++--- > gcc/testsuite/gcc.target/i386/pr89229-4a.c | 4 +--- > 2 files changed, 18 insertions(+), 6 deletions(-) > > diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md > index c906d75b13e..77182e34fe1 100644 > --- a/gcc/config/i386/i386.md > +++ b/gcc/config/i386/i386.md > @@ -2961,8 +2961,12 @@ (define_insn "*movhi_internal" > ] > (const_string "TI")) > (eq_attr "alternative" "12") > - (cond [(match_test "TARGET_AVX512FP16") > + (cond [(match_test "TARGET_AVX512VL") > + (const_string "TI") > + (match_test "TARGET_AVX512FP16") > (const_string "HF") > + (match_test "TARGET_AVX512F") > + (const_string "SF") > (match_test "TARGET_AVX") > (const_string "TI") > (ior (not (match_test "TARGET_SSE2")) > @@ -4099,8 +4103,12 @@ (define_insn "*movdf_internal" > > /* movaps is one byte shorter for non-AVX targets. */ > (eq_attr "alternative" "13,17") > - (cond [(match_test "TARGET_AVX") > + (cond [(match_test "TARGET_AVX512VL") > + (const_string "V2DF") > + (match_test "TARGET_AVX512F") > (const_string "DF") > + (match_test "TARGET_AVX") > + (const_string "V2DF") > (ior (not (match_test "TARGET_SSE2")) > (match_test "optimize_function_for_size_p (c= fun)")) > (const_string "V4SF") > @@ -4380,8 +4388,14 @@ (define_insn "*mov_internal" > (const_string "HI") > (const_string "TI")) > (eq_attr "alternative" "5") > - (cond [(match_test "TARGET_AVX512FP16") > + (cond [(match_test "TARGET_AVX512VL") > + (const_string "V4SF") > + (match_test "TARGET_AVX512FP16") > (const_string "HF") > + (match_test "TARGET_AVX512F") > + (const_string "SF") > + (match_test "TARGET_AVX") > + (const_string "V4SF") > (ior (match_test "TARGET_SSE_PARTIAL_REG_DEPENDEN= CY") > (match_test "TARGET_SSE_SPLIT_REGS")) > (const_string "V4SF") > diff --git a/gcc/testsuite/gcc.target/i386/pr89229-4a.c b/gcc/testsuite/g= cc.target/i386/pr89229-4a.c > index 5bc10d25619..8869650b0ad 100644 > --- a/gcc/testsuite/gcc.target/i386/pr89229-4a.c > +++ b/gcc/testsuite/gcc.target/i386/pr89229-4a.c > @@ -1,4 +1,4 @@ > -/* { dg-do compile { target { ! ia32 } } } */ > +/* { dg-do assemble { target { ! ia32 } } } */ > /* { dg-options "-O2 -march=3Dskylake-avx512" } */ > > extern double d; > @@ -12,5 +12,3 @@ foo1 (double x) > asm volatile ("" : "+v" (xmm17)); > d =3D xmm17; > } > - > -/* { dg-final { scan-assembler-not "vmovapd" } } */ > -- > 2.31.1 > --=20 BR, Hongtao