From: Kito Cheng <kito.cheng@sifive.com>
Date: Tue, 2 Dec 2025 13:40:06 +0000 (-0700)
Subject: RISC-V: Remove unused placeholder_p parameter from add_function
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=24a1085072b034b1cdf8c952fbfbacbccd44b7e9;p=thirdparty%2Fgcc.git

RISC-V: Remove unused placeholder_p parameter from add_function

The placeholder_p parameter of function_builder::add_function is always
passed as false.  This was inherited from the AArch64 implementation but
is unnecessary for RISC-V.

gcc/ChangeLog:

	* config/riscv/riscv-vector-builtins.cc
	(function_builder::add_function): Remove placeholder_p parameter.
	(function_builder::add_unique_function): Update call.
	(function_builder::add_overloaded_function): Likewise.
	* config/riscv/riscv-vector-builtins.h
	(function_builder::add_function): Update declaration.
---

diff --git a/gcc/config/riscv/riscv-vector-builtins.cc b/gcc/config/riscv/riscv-vector-builtins.cc
index a3e596a8097..d37cc43ebe5 100644
--- a/gcc/config/riscv/riscv-vector-builtins.cc
+++ b/gcc/config/riscv/riscv-vector-builtins.cc
@@ -4443,7 +4443,7 @@ function_builder::get_attributes (const function_instance &instance)
 registered_function &
 function_builder::add_function (const function_instance &instance,
 				const char *name, tree fntype, tree attrs,
-				bool placeholder_p, const char *overload_name,
+				const char *overload_name,
 				const vec<tree> &argument_types,
 				enum required_ext required,
 				bool overloaded_p = false)
@@ -4464,7 +4464,7 @@ function_builder::add_function (const function_instance &instance,
      nodes and remove the target hook. For now, however, we need to appease the
      validation and return a non-NULL, non-error_mark_node node, so we
      arbitrarily choose integer_zero_node.  */
-  tree decl = placeholder_p || in_lto_p
+  tree decl = in_lto_p
 		? integer_zero_node
 		: simulate_builtin_function_decl (input_location, name, fntype,
 						  code, NULL, attrs);
@@ -4508,7 +4508,7 @@ function_builder::add_unique_function (const function_instance &instance,
 				 argument_types.address ());
   tree attrs = get_attributes (instance);
   registered_function &rfn
-    = add_function (instance, name, fntype, attrs, false, overload_name,
+    = add_function (instance, name, fntype, attrs, overload_name,
 		    argument_types.copy (), required);
 
   /* Enter the function into the hash table.  */
@@ -4523,7 +4523,7 @@ function_builder::add_unique_function (const function_instance &instance,
       /* Attribute lists shouldn't be shared.  */
       tree attrs = get_attributes (instance);
       if (m_direct_overloads)
-	add_function (instance, overload_name, fntype, attrs, false, NULL,
+	add_function (instance, overload_name, fntype, attrs, NULL,
 		      vNULL, required);
       else
 	{
@@ -4562,7 +4562,7 @@ function_builder::add_overloaded_function (const function_instance &instance,
       /* To avoid API conflicting, take void return type and void argument
 	 for the overloaded function.  */
       tree fntype = build_function_type (void_type_node, void_list_node);
-      add_function (instance, name, fntype, NULL_TREE, false, name,
+      add_function (instance, name, fntype, NULL_TREE, name,
 		    vNULL, required, true);
       obstack_free (&m_string_obstack, name);
     }
diff --git a/gcc/config/riscv/riscv-vector-builtins.h b/gcc/config/riscv/riscv-vector-builtins.h
index 8cb243ba53b..9e1a47499ba 100644
--- a/gcc/config/riscv/riscv-vector-builtins.h
+++ b/gcc/config/riscv/riscv-vector-builtins.h
@@ -435,7 +435,7 @@ private:
   tree get_attributes (const function_instance &);
 
   registered_function &add_function (const function_instance &, const char *,
-				     tree, tree, bool, const char *,
+				     tree, tree, const char *,
 				     const vec<tree> &, enum required_ext,
 				     bool);