From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1130) id 2F65A3857016; Mon, 5 Sep 2022 13:14:58 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 2F65A3857016 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1662383698; bh=vTBazFlCTCvu3adtdXjwVob8V4PkLkkT2N8A1lOAN0Y=; h=From:To:Subject:Date:From; b=Owyskrss3VD9qDaRPpIDKWM/DmYyoAV3QByLzjwI2d7Rw3gzzZSzRbbUZiNmKIh9W HzVvCTPXpH+Rj3TtnqmGGOjEAt9p14SyseUtMAOq/16ex0HQc/Ie+/gnCxw3i1TmOD CxXuNY0Idv42u9Nyd8TfijQLQ5DwJ5QqVCiplzMw= MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Richard Sandiford To: gcc-cvs@gcc.gnu.org Subject: [gcc r13-2434] aarch64: Remove lazy SIMD builtin initialisation X-Act-Checkin: gcc X-Git-Author: Richard Sandiford X-Git-Refname: refs/heads/trunk X-Git-Oldrev: 7742b797b3fd7acea28a0a0bff9bbb06e7c2d0ea X-Git-Newrev: d29ecf23f0b046de720a25171c3b3980fa8a0a22 Message-Id: <20220905131458.2F65A3857016@sourceware.org> Date: Mon, 5 Sep 2022 13:14:58 +0000 (GMT) List-Id: https://gcc.gnu.org/g:d29ecf23f0b046de720a25171c3b3980fa8a0a22 commit r13-2434-gd29ecf23f0b046de720a25171c3b3980fa8a0a22 Author: Richard Sandiford Date: Mon Sep 5 14:14:44 2022 +0100 aarch64: Remove lazy SIMD builtin initialisation At one time the aarch64 port registered the Advanced SIMD builtins lazily, when we first encountered a set of target flags that includes +simd. These days we always initialise them at start-up, temporarily forcing a conducive set of flags if necessary. This patch removes some vestiges of the old way of doing things. gcc/ * config/aarch64/aarch64-protos.h (aarch64_init_simd_builtins): Remove prototype. * config/aarch64/aarch64-builtins.cc (aarch64_simd_builtins_initialized_p): Delete. (aarch64_init_simd_builtins): Make static. Remove protection against multiple calls. * config/aarch64/aarch64-c.cc (aarch64_pragma_target_parse): Remove lazy SIMD builtin initialization. * config/aarch64/aarch64.cc (aarch64_option_valid_attribute_p): Likewise. Diff: --- gcc/config/aarch64/aarch64-builtins.cc | 9 +-------- gcc/config/aarch64/aarch64-c.cc | 13 ------------- gcc/config/aarch64/aarch64-protos.h | 1 - gcc/config/aarch64/aarch64.cc | 12 ------------ 4 files changed, 1 insertion(+), 34 deletions(-) diff --git a/gcc/config/aarch64/aarch64-builtins.cc b/gcc/config/aarch64/aarch64-builtins.cc index f90fda4ca97..5eef5aaa402 100644 --- a/gcc/config/aarch64/aarch64-builtins.cc +++ b/gcc/config/aarch64/aarch64-builtins.cc @@ -1300,8 +1300,6 @@ aarch64_get_attributes (unsigned int f, machine_mode mode) return aarch64_add_attribute ("leaf", attrs); } -static bool aarch64_simd_builtins_initialized_p = false; - /* Due to the architecture not providing lane variant of the lane instructions for fcmla we can't use the standard simd builtin expansion code, but we still want the majority of the validation that would normally be done. */ @@ -1586,14 +1584,9 @@ handle_arm_neon_h (void) aarch64_init_simd_intrinsics (); } -void +static void aarch64_init_simd_builtins (void) { - if (aarch64_simd_builtins_initialized_p) - return; - - aarch64_simd_builtins_initialized_p = true; - aarch64_init_simd_builtin_types (); /* Strong-typing hasn't been implemented for all AdvSIMD builtin intrinsics. diff --git a/gcc/config/aarch64/aarch64-c.cc b/gcc/config/aarch64/aarch64-c.cc index 3d2fb5ec2ef..52ed4a218a8 100644 --- a/gcc/config/aarch64/aarch64-c.cc +++ b/gcc/config/aarch64/aarch64-c.cc @@ -270,19 +270,6 @@ aarch64_pragma_target_parse (tree args, tree pop_target) if (pop_target) aarch64_save_restore_target_globals (pop_target); - /* Initialize SIMD builtins if we haven't already. - Set current_target_pragma to NULL for the duration so that - the builtin initialization code doesn't try to tag the functions - being built with the attributes specified by any current pragma, thus - going into an infinite recursion. */ - if (TARGET_SIMD) - { - tree saved_current_target_pragma = current_target_pragma; - current_target_pragma = NULL; - aarch64_init_simd_builtins (); - current_target_pragma = saved_current_target_pragma; - } - return true; } diff --git a/gcc/config/aarch64/aarch64-protos.h b/gcc/config/aarch64/aarch64-protos.h index 99af10ad534..5ecdb8af863 100644 --- a/gcc/config/aarch64/aarch64-protos.h +++ b/gcc/config/aarch64/aarch64-protos.h @@ -907,7 +907,6 @@ void aarch64_sve_expand_vector_init (rtx, rtx); void aarch64_init_cumulative_args (CUMULATIVE_ARGS *, const_tree, rtx, const_tree, unsigned, bool = false); void aarch64_init_expanders (void); -void aarch64_init_simd_builtins (void); void aarch64_emit_call_insn (rtx); void aarch64_register_pragmas (void); void aarch64_relayout_simd_types (void); diff --git a/gcc/config/aarch64/aarch64.cc b/gcc/config/aarch64/aarch64.cc index b12f13f4f22..2311ad0e2b8 100644 --- a/gcc/config/aarch64/aarch64.cc +++ b/gcc/config/aarch64/aarch64.cc @@ -18898,18 +18898,6 @@ aarch64_option_valid_attribute_p (tree fndecl, tree, tree args, int) if (ret) { aarch64_override_options_internal (&global_options); - /* Initialize SIMD builtins if we haven't already. - Set current_target_pragma to NULL for the duration so that - the builtin initialization code doesn't try to tag the functions - being built with the attributes specified by any current pragma, thus - going into an infinite recursion. */ - if (TARGET_SIMD) - { - tree saved_current_target_pragma = current_target_pragma; - current_target_pragma = NULL; - aarch64_init_simd_builtins (); - current_target_pragma = saved_current_target_pragma; - } new_target = build_target_option_node (&global_options, &global_options_set); }