[1/2] Remove unused remove_node_from_expr_list

Message ID 20220719151532.8206-1-amonakov@ispras.ru
State New, archived
Headers
Series [1/2] Remove unused remove_node_from_expr_list |

Commit Message

Li, Pan2 via Gcc-patches July 19, 2022, 3:15 p.m. UTC
  This function remains unused since remove_node_from_insn_list was cloned
from it.

gcc/ChangeLog:

	* rtl.h (remove_node_from_expr_list): Remove declaration.
	* rtlanal.cc (remove_node_from_expr_list): Remove (no uses).
---
 gcc/rtl.h      |  1 -
 gcc/rtlanal.cc | 29 -----------------------------
 2 files changed, 30 deletions(-)
  

Comments

Li, Pan2 via Gcc-patches July 20, 2022, 7:34 a.m. UTC | #1
On Tue, Jul 19, 2022 at 5:16 PM Alexander Monakov via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> This function remains unused since remove_node_from_insn_list was cloned
> from it.

OK.

> gcc/ChangeLog:
>
>         * rtl.h (remove_node_from_expr_list): Remove declaration.
>         * rtlanal.cc (remove_node_from_expr_list): Remove (no uses).
> ---
>  gcc/rtl.h      |  1 -
>  gcc/rtlanal.cc | 29 -----------------------------
>  2 files changed, 30 deletions(-)
>
> diff --git a/gcc/rtl.h b/gcc/rtl.h
> index 488016bb4..645c009a3 100644
> --- a/gcc/rtl.h
> +++ b/gcc/rtl.h
> @@ -3712,7 +3712,6 @@ extern unsigned hash_rtx_cb (const_rtx, machine_mode, int *, int *,
>  extern rtx regno_use_in (unsigned int, rtx);
>  extern int auto_inc_p (const_rtx);
>  extern bool in_insn_list_p (const rtx_insn_list *, const rtx_insn *);
> -extern void remove_node_from_expr_list (const_rtx, rtx_expr_list **);
>  extern void remove_node_from_insn_list (const rtx_insn *, rtx_insn_list **);
>  extern int loc_mentioned_in_p (rtx *, const_rtx);
>  extern rtx_insn *find_first_parameter_load (rtx_insn *, rtx_insn *);
> diff --git a/gcc/rtlanal.cc b/gcc/rtlanal.cc
> index d78cc6024..ec95ecd6c 100644
> --- a/gcc/rtlanal.cc
> +++ b/gcc/rtlanal.cc
> @@ -2878,35 +2878,6 @@ in_insn_list_p (const rtx_insn_list *listp, const rtx_insn *node)
>    return false;
>  }
>
> -/* Search LISTP (an EXPR_LIST) for an entry whose first operand is NODE and
> -   remove that entry from the list if it is found.
> -
> -   A simple equality test is used to determine if NODE matches.  */
> -
> -void
> -remove_node_from_expr_list (const_rtx node, rtx_expr_list **listp)
> -{
> -  rtx_expr_list *temp = *listp;
> -  rtx_expr_list *prev = NULL;
> -
> -  while (temp)
> -    {
> -      if (node == temp->element ())
> -       {
> -         /* Splice the node out of the list.  */
> -         if (prev)
> -           XEXP (prev, 1) = temp->next ();
> -         else
> -           *listp = temp->next ();
> -
> -         return;
> -       }
> -
> -      prev = temp;
> -      temp = temp->next ();
> -    }
> -}
> -
>  /* Search LISTP (an INSN_LIST) for an entry whose first operand is NODE and
>     remove that entry from the list if it is found.
>
> --
> 2.35.1
>
  

Patch

diff --git a/gcc/rtl.h b/gcc/rtl.h
index 488016bb4..645c009a3 100644
--- a/gcc/rtl.h
+++ b/gcc/rtl.h
@@ -3712,7 +3712,6 @@  extern unsigned hash_rtx_cb (const_rtx, machine_mode, int *, int *,
 extern rtx regno_use_in (unsigned int, rtx);
 extern int auto_inc_p (const_rtx);
 extern bool in_insn_list_p (const rtx_insn_list *, const rtx_insn *);
-extern void remove_node_from_expr_list (const_rtx, rtx_expr_list **);
 extern void remove_node_from_insn_list (const rtx_insn *, rtx_insn_list **);
 extern int loc_mentioned_in_p (rtx *, const_rtx);
 extern rtx_insn *find_first_parameter_load (rtx_insn *, rtx_insn *);
diff --git a/gcc/rtlanal.cc b/gcc/rtlanal.cc
index d78cc6024..ec95ecd6c 100644
--- a/gcc/rtlanal.cc
+++ b/gcc/rtlanal.cc
@@ -2878,35 +2878,6 @@  in_insn_list_p (const rtx_insn_list *listp, const rtx_insn *node)
   return false;
 }
 
-/* Search LISTP (an EXPR_LIST) for an entry whose first operand is NODE and
-   remove that entry from the list if it is found.
-
-   A simple equality test is used to determine if NODE matches.  */
-
-void
-remove_node_from_expr_list (const_rtx node, rtx_expr_list **listp)
-{
-  rtx_expr_list *temp = *listp;
-  rtx_expr_list *prev = NULL;
-
-  while (temp)
-    {
-      if (node == temp->element ())
-	{
-	  /* Splice the node out of the list.  */
-	  if (prev)
-	    XEXP (prev, 1) = temp->next ();
-	  else
-	    *listp = temp->next ();
-
-	  return;
-	}
-
-      prev = temp;
-      temp = temp->next ();
-    }
-}
-
 /* Search LISTP (an INSN_LIST) for an entry whose first operand is NODE and
    remove that entry from the list if it is found.