From patchwork Thu May 4 00:44:15 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans-Peter Nilsson X-Patchwork-Id: 1776685 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=gcc.gnu.org (client-ip=2620:52:3:1:0:246e:9693:128c; helo=sourceware.org; envelope-from=gcc-patches-bounces+incoming=patchwork.ozlabs.org@gcc.gnu.org; receiver=) Authentication-Results: legolas.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.a=rsa-sha256 header.s=default header.b=jDYwPPPy; dkim-atps=neutral Received: from sourceware.org (server2.sourceware.org [IPv6:2620:52:3:1:0:246e:9693:128c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4QBZpJ0wsgz1ydV for ; Thu, 4 May 2023 10:44:38 +1000 (AEST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id E45E9385772D for ; Thu, 4 May 2023 00:44:35 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E45E9385772D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1683161075; bh=PAIJeMuufUUbhlMkGbuUqF2nEBbkllhIIItFa4nMi5Y=; h=To:Subject:Date:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:From; b=jDYwPPPy9BjMQj9PoLJbuVAlbAjrCsbbLLPMABwmHAc4KtXNNXFJFBhlPYc8tQU5j ihmRLngjYW1/t5453NvaHyS3KZnGbopsG25VIKvO2xWVH1el1Wvg7aISL9AogQgbHM tgT47/A3wNK/A0OeBShjwXEQvYCStvCJMSOcv1GI= X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from smtp1.axis.com (smtp1.axis.com [195.60.68.17]) by sourceware.org (Postfix) with ESMTPS id 735853858D28 for ; Thu, 4 May 2023 00:44:16 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 735853858D28 To: Subject: [committed] CRIS-LRA: Define TARGET_SPILL_CLASS MIME-Version: 1.0 Message-ID: <20230504004415.3F59E20438@pchp3.se.axis.com> Date: Thu, 4 May 2023 02:44:15 +0200 X-Spam-Status: No, score=-11.3 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_PASS, 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 X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Hans-Peter Nilsson via Gcc-patches From: Hans-Peter Nilsson Reply-To: Hans-Peter Nilsson Errors-To: gcc-patches-bounces+incoming=patchwork.ozlabs.org@gcc.gnu.org Sender: "Gcc-patches" This has no effect on arith-rand-ll (which suffers badly from LRA) and marginal effects (0.01% improvement) on coremark, but the size of coremark shrinks by 0.2%. An earlier version was tested with a tree around 2023-03 which showed (marginally) that ALL_REGS is preferable to GENERAL_REGS. * config/cris/cris.cc (TARGET_SPILL_CLASS): Define to ALL_REGS. --- gcc/config/cris/cris.cc | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/gcc/config/cris/cris.cc b/gcc/config/cris/cris.cc index 641e7ea25fb1..373b9999e0db 100644 --- a/gcc/config/cris/cris.cc +++ b/gcc/config/cris/cris.cc @@ -134,6 +134,7 @@ static void cris_init_libfuncs (void); static unsigned int cris_postdbr_cmpelim (void); static reg_class_t cris_preferred_reload_class (rtx, reg_class_t); +static reg_class_t cris_spill_class (reg_class_t, machine_mode); static int cris_register_move_cost (machine_mode, reg_class_t, reg_class_t); static int cris_memory_move_cost (machine_mode, reg_class_t, bool); @@ -224,6 +225,9 @@ int cris_cpu_version = CRIS_DEFAULT_CPU_VERSION; #undef TARGET_PREFERRED_RELOAD_CLASS #define TARGET_PREFERRED_RELOAD_CLASS cris_preferred_reload_class +#undef TARGET_SPILL_CLASS +#define TARGET_SPILL_CLASS cris_spill_class + /* We don't define TARGET_FIXED_CONDITION_CODE_REGS, as at the time of this writing, it has an effect only on pre-reload CSE and when scheduling (and for "macro fusion" at that). Neither applies for @@ -1684,6 +1688,14 @@ cris_preferred_reload_class (rtx x, reg_class_t rclass) return rclass; } +/* Worker function for TARGET_SPILL_CLASS. */ + +static reg_class_t +cris_spill_class (reg_class_t /* orig_class */, machine_mode) +{ + return ALL_REGS; +} + /* Worker function for TARGET_REGISTER_MOVE_COST. */ static int