[gcc(refs/vendors/riscv/heads/gcc-15-with-riscv-opts)] RISC-V: Remove unused placeholder_p parameter from add_function

Jeff Law law@gcc.gnu.org
Thu Jan 29 19:48:31 GMT 2026


https://gcc.gnu.org/g:8e5098b7562386c601bbe426e3432b99b9c8d090

commit 8e5098b7562386c601bbe426e3432b99b9c8d090
Author: Kito Cheng <kito.cheng@sifive.com>
Date:   Tue Dec 2 06:40:06 2025 -0700

    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.
    
    (cherry picked from commit 24a1085072b034b1cdf8c952fbfbacbccd44b7e9)

Diff:
---
 gcc/config/riscv/riscv-vector-builtins.cc | 10 +++++-----
 gcc/config/riscv/riscv-vector-builtins.h  |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/gcc/config/riscv/riscv-vector-builtins.cc b/gcc/config/riscv/riscv-vector-builtins.cc
index c22b751badb2..f92e94b781d7 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 8cb243ba53b0..9e1a47499ba6 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);


More information about the Gcc-cvs mailing list