From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk1-x72f.google.com (mail-qk1-x72f.google.com [IPv6:2607:f8b0:4864:20::72f]) by sourceware.org (Postfix) with ESMTPS id 23C953858C83 for ; Sun, 23 Apr 2023 21:06:54 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 23C953858C83 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-qk1-x72f.google.com with SMTP id af79cd13be357-74ab718c344so1058955785a.1 for ; Sun, 23 Apr 2023 14:06:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1682284013; x=1684876013; 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=0JZCndpMyVogdYkL1mQuYHQYiizp9X4E3Roa+7Zih1A=; b=kLC19bJ4XwkgqQ2GGscl9E3kZu+hErvL2azYFaE7xIBMiCnreF+ZpYw59HaHdzYvOO 4Rczq7jZXra0XxDRLHvmwHYAcHEurYTeufi0YgQLBhdOLAO0RgfTtSj7bpmaTgDS5CsB M0VujXaTWEYBKFBinEjXblFXPB+79kDPUU+CqC2YG1XET8H1DMggtlRfTLQUmbGpFrVN YzNgcHp9kIKmPzItXYz0eXXwS6QO+06k+nYiQKI9+vfpXq7c6QToBGxjfggziJNKDtgx pxPxeFv7zRprvpi6kjcnsgs/AENeCSmvKo7ODSvxKhtQT2LmovWQ1YRSietLDSOondru /4AQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1682284013; x=1684876013; 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=0JZCndpMyVogdYkL1mQuYHQYiizp9X4E3Roa+7Zih1A=; b=cTvXicwBBlWZKnZJFFjRDSXLa7OSlJincm6MkOiaB0x18KEAq8A2sS4fFeNnhV1x8s xmM1vPLdZNnNSbY5MI9neuhW9eWXRKv01Pa2YdX/Y91ml0sS/UY/bBkQQ9htSGxWMWwK SOr9E6y+Yz6+5T0/s3OxWk3pMcFzvzfhtJ6gYnUnDNN7xP7YeinosYeyXkPDqV2xbHYd Hq8LWdLEGQDcx0lg4yO4XvOMH8AauanX9gkRMArU/NrCzT384RELLvcHw7bquf+0d+N9 OiioutMRZN8MmO+MtPNGX0VA6vcptmrFs8oEwEF4JDS1EFG0zeWMgvSDnyu2HnrOkH6c DPmg== X-Gm-Message-State: AAQBX9fHjg/RlEHn527QtfUUGkK83eLeaMWMITNxojSNiC51Uj0bK9D7 phZlT2OWjyDwoLnTjI9H10E+OmWre0HhpJxemxAKNRlaOGZBlw== X-Google-Smtp-Source: AKy350Z27xm3kzoWvqqR4NS7QdV88/Ca2yYSBYhsORUacHCbnufWdMlc/2ilSGlx1sYRDEGzwZ6CS5hWhZLKDcme4Jo= X-Received: by 2002:ad4:5dc2:0:b0:5e8:f869:37fd with SMTP id m2-20020ad45dc2000000b005e8f86937fdmr18792547qvh.8.1682284013224; Sun, 23 Apr 2023 14:06:53 -0700 (PDT) MIME-Version: 1.0 References: <283c45ca085ced958cbce6e64331252c83a5899f.1682268126.git.segher@kernel.crashing.org> In-Reply-To: <283c45ca085ced958cbce6e64331252c83a5899f.1682268126.git.segher@kernel.crashing.org> From: Uros Bizjak Date: Sun, 23 Apr 2023 23:06:41 +0200 Message-ID: Subject: Re: [PATCH] Turn on LRA on all targets To: Segher Boessenkool Cc: gcc-patches@gcc.gnu.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-2.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,KAM_SHORT,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE 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: On Sun, Apr 23, 2023 at 6:48=E2=80=AFPM Segher Boessenkool wrote: > > This minimal patch enables LRA for all targets. It does not clean up > the target code, nor does it do anything to generic code: it just > deletes all target definitions of TARGET_LRA_P. > > There are three kinds of changes: > > 1) Targets that already always have LRA, but that redefine the hook > anyway. These are gcn, pdp11, rx, sparc, vax, and xtensa. Nothing > really changes for these targets with this patch (but later patches > will delete the superfluous hook implementations). > 2) Targets that have LRA selectable. Some of those are probably fine, > since they default to using LRA (arc, mips, s390). Two others don't > though, maybe because there are problems (ft32 and sh). I'd love to > hear from all targets in this category what the status is, how easy it > was to convert, etc. > 3) Targets that as of yet never used LRA. Many of those will be fine, > but some others will need a little tuning, and a few might need some > actual improvements to LRA itself. These are cris, epiphany, fr30, > frv, h8300, ia64, iq2000, lm32, m32c, m32r, m68k, mcore, microblaze, > mmix, mn10300, msp430, nvptx, pa, rl78, stormy16, and visium. We'll > find out how many of those targets are ever tested, and how many of > those work with LRA without further changes, and how well. > > I send this patch now so that people can start testing. I don't plan to > commit this for another week at least, for a week after GCC 13 release I > guess? How does that plan sound to people? An old patch to enable Alpha is at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D66207 Uros.