From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <richard.guenther@gmail.com>
Received: from mail-ej1-x62d.google.com (mail-ej1-x62d.google.com
 [IPv6:2a00:1450:4864:20::62d])
 by sourceware.org (Postfix) with ESMTPS id 2933E3858003
 for <gcc-patches@gcc.gnu.org>; Mon, 13 Sep 2021 09:38:48 +0000 (GMT)
DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 2933E3858003
Received: by mail-ej1-x62d.google.com with SMTP id o20so8968163ejd.7
 for <gcc-patches@gcc.gnu.org>; Mon, 13 Sep 2021 02:38:48 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20210112;
 h=x-gm-message-state:mime-version:references:in-reply-to:from:date
 :message-id:subject:to:cc;
 bh=1h3sVXlDHY6EE2JzL4ZaXucaO9Hff49ygNxD7AWgnBQ=;
 b=1KhXYdQYAm/2ZuJRHJwAgwVRPVEjZSnDNXvAnaJ5y0h/PVNJMN6e2RUEEg6y3ZS1qJ
 lhXB1RDjBBzB+b7fepyoDEF8Wq50oX0rfBXgFv31vmmWE+DekKgXTpzd4GpqbTNV//E6
 c9hAXgbY1qBxrfd3sz0QG9aDuvITI/PSV+MKqwsSciteXih88e4L488HEVjFSpVle+bC
 wWqr8xNgHKZ+eKlrZ2OCP3UAxXpqpaqSDU3rJV09tNzbYa23iZDuzbGo5mk7fQnLQc7p
 CwfauLE9oq79MrGjxGv7y6Bx8qS/2wKk8NsHx9rURGw0E9uXS8ftHp4ovluCPIb9uVBf
 7Q8g==
X-Gm-Message-State: AOAM531OPmkZSSTp0/CE7W125R9F6P/SckXSuJ304bqAHIjyl28TmWLv
 pCcQz9HH51JHqBIUuXiCMHavuwPrx7iYg7Sbl+yLf5a/
X-Google-Smtp-Source: ABdhPJwmbayMSx9ohcDao1yoNNQ7RWIx3kWRd1pZ5IFxzq29FVbH3Zhp1B4nYivIVbMnnjtWXjcB4ocTypn5/77efrU=
X-Received: by 2002:a17:906:ece1:: with SMTP id
 qt1mr11391080ejb.281.1631525927195; 
 Mon, 13 Sep 2021 02:38:47 -0700 (PDT)
MIME-Version: 1.0
References: <20210910144841.3139174-1-rearnsha@arm.com>
 <20210910144841.3139174-2-rearnsha@arm.com>
In-Reply-To: <20210910144841.3139174-2-rearnsha@arm.com>
From: Richard Biener <richard.guenther@gmail.com>
Date: Mon, 13 Sep 2021 11:38:36 +0200
Message-ID: <CAFiYyc2XBdHGD=rpGcbm-Szce2hHtgmwUrWPd=84Ww=8UWoR7Q@mail.gmail.com>
Subject: Re: [PATCH v3 1/3] rtl: directly handle MEM in gen_highpart [PR102125]
To: Richard Earnshaw <rearnsha@arm.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>
Content-Type: text/plain; charset="UTF-8"
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00, DKIM_SIGNED,
 DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,
 SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4
X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on
 server2.sourceware.org
X-BeenThere: gcc-patches@gcc.gnu.org
X-Mailman-Version: 2.1.29
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>
X-List-Received-Date: Mon, 13 Sep 2021 09:38:49 -0000

On Fri, Sep 10, 2021 at 4:48 PM Richard Earnshaw <rearnsha@arm.com> wrote:
>
>
> gen_lowpart_general handles forming a lowpart of a MEM by using
> adjust_address to rework and validate a new version of the MEM.
> Do the same for gen_highpart rather than calling simplify_gen_subreg
> for this case.

OK from my side.

Thanks,
Richard.

> gcc/ChangeLog:
>
>         PR target/102125
>         * emit-rtl.c (gen_highpart): Use adjust_address to handle
>         MEM rather than calling simplify_gen_subreg.
> ---
>  gcc/emit-rtl.c | 23 +++++++++++++----------
>  1 file changed, 13 insertions(+), 10 deletions(-)
>