Patchwork Remove unused ivtype_map symbols from sese.[hc]

login
register
mail settings
Submitter Lawrence Crowl
Date March 31, 2013, 10:19 p.m.
Message ID <CAGqM8fYWy-z7oesChP+eOr1EEs7nsBcGPcr2c4r5v4edbDLbyQ@mail.gmail.com>
Download mbox | patch
Permalink /patch/232633/
State New
Headers show

Comments

Lawrence Crowl - March 31, 2013, 10:19 p.m.
Remove unused symbols related to ivtype_map.  This map does not appear to
exist and I see no evidence of its removal in the ChangeLog.

Tested on x86_64.

Okay for trunk?
Richard Guenther - April 2, 2013, 9:40 a.m.
On Mon, Apr 1, 2013 at 12:19 AM, Lawrence Crowl <crowl@googlers.com> wrote:
> Remove unused symbols related to ivtype_map.  This map does not appear to
> exist and I see no evidence of its removal in the ChangeLog.
>
> Tested on x86_64.
>
> Okay for trunk?

Ok.

Thanks,
Richard.

>
> Index: gcc/ChangeLog
>
> 2013-03-31  Lawrence Crowl  <crowl@google.com>
>
>         * sese.h (struct ivtype_map_elt_s): Remove unused.
>         (extern debug_ivtype_map): Remove unused.
>         (extern eq_ivtype_map_elts): Remove unused.
>         * sese.c (debug_ivtype_map): Removed unused.
>         (debug_ivtype_map_1): Removed unused.
>         (debug_ivtype_elt): Remove unused.
>         (eq_ivtype_map_elts): Remove unused.
>
>
> Index: gcc/sese.c
> ===================================================================
> --- gcc/sese.c  (revision 197272)
> +++ gcc/sese.c  (working copy)
> @@ -83,47 +83,6 @@ eq_rename_map_elts (const void *e1, cons
>
>
>
> -/* Print to stderr the element ELT.  */
> -
> -static void
> -debug_ivtype_elt (ivtype_map_elt elt)
> -{
> -  fprintf (stderr, "(%s, ", elt->cloog_iv);
> -  print_generic_expr (stderr, elt->type, 0);
> -  fprintf (stderr, ")\n");
> -}
> -
> -/* Helper function for debug_ivtype_map.  */
> -
> -static int
> -debug_ivtype_map_1 (void **slot, void *s ATTRIBUTE_UNUSED)
> -{
> -  struct ivtype_map_elt_s *entry = (struct ivtype_map_elt_s *) *slot;
> -  debug_ivtype_elt (entry);
> -  return 1;
> -}
> -
> -/* Print to stderr all the elements of MAP.  */
> -
> -DEBUG_FUNCTION void
> -debug_ivtype_map (htab_t map)
> -{
> -  htab_traverse (map, debug_ivtype_map_1, NULL);
> -}
> -
> -/* Compares database elements E1 and E2.  */
> -
> -int
> -eq_ivtype_map_elts (const void *e1, const void *e2)
> -{
> -  const struct ivtype_map_elt_s *elt1 = (const struct ivtype_map_elt_s *) e1;
> -  const struct ivtype_map_elt_s *elt2 = (const struct ivtype_map_elt_s *) e2;
> -
> -  return (elt1->cloog_iv == elt2->cloog_iv);
> -}
> -
> -
> -
>  /* Record LOOP as occurring in REGION.  */
>
>  static void
> Index: gcc/sese.h
> ===================================================================
> --- gcc/sese.h  (revision 197272)
> +++ gcc/sese.h  (working copy)
> @@ -275,17 +275,6 @@ new_rename_map_elt (tree old_name, tree
>    return res;
>  }
>
> -/* Structure containing the mapping between the CLooG's induction
> -   variable and the type of the old induction variable.  */
> -typedef struct ivtype_map_elt_s
> -{
> -  tree type;
> -  const char *cloog_iv;
> -} *ivtype_map_elt;
> -
> -extern void debug_ivtype_map (htab_t);
> -extern int eq_ivtype_map_elts (const void *, const void *);
> -
>  /* Free and compute again all the dominators information.  */
>
>  static inline void
>
> --
> Lawrence Crowl
Lawrence Crowl - April 2, 2013, 8:14 p.m.
On 4/2/13, Richard Biener <richard.guenther@gmail.com> wrote:
> On Mon, Apr 1, 2013 at 12:19 AM, Lawrence Crowl <crowl@googlers.com> wrote:
>> Remove unused symbols related to ivtype_map.  This map does not appear to
>> exist and I see no evidence of its removal in the ChangeLog.
>>
>> Tested on x86_64.
>>
>> Okay for trunk?
>
> Ok.

Committed.

Patch

Index: gcc/ChangeLog

2013-03-31  Lawrence Crowl  <crowl@google.com>

	* sese.h (struct ivtype_map_elt_s): Remove unused.
	(extern debug_ivtype_map): Remove unused.
	(extern eq_ivtype_map_elts): Remove unused.
	* sese.c (debug_ivtype_map): Removed unused.
	(debug_ivtype_map_1): Removed unused.
	(debug_ivtype_elt): Remove unused.
	(eq_ivtype_map_elts): Remove unused.


Index: gcc/sese.c
===================================================================
--- gcc/sese.c	(revision 197272)
+++ gcc/sese.c	(working copy)
@@ -83,47 +83,6 @@  eq_rename_map_elts (const void *e1, cons

 

-/* Print to stderr the element ELT.  */
-
-static void
-debug_ivtype_elt (ivtype_map_elt elt)
-{
-  fprintf (stderr, "(%s, ", elt->cloog_iv);
-  print_generic_expr (stderr, elt->type, 0);
-  fprintf (stderr, ")\n");
-}
-
-/* Helper function for debug_ivtype_map.  */
-
-static int
-debug_ivtype_map_1 (void **slot, void *s ATTRIBUTE_UNUSED)
-{
-  struct ivtype_map_elt_s *entry = (struct ivtype_map_elt_s *) *slot;
-  debug_ivtype_elt (entry);
-  return 1;
-}
-
-/* Print to stderr all the elements of MAP.  */
-
-DEBUG_FUNCTION void
-debug_ivtype_map (htab_t map)
-{
-  htab_traverse (map, debug_ivtype_map_1, NULL);
-}
-
-/* Compares database elements E1 and E2.  */
-
-int
-eq_ivtype_map_elts (const void *e1, const void *e2)
-{
-  const struct ivtype_map_elt_s *elt1 = (const struct ivtype_map_elt_s *) e1;
-  const struct ivtype_map_elt_s *elt2 = (const struct ivtype_map_elt_s *) e2;
-
-  return (elt1->cloog_iv == elt2->cloog_iv);
-}
-
-
-
 /* Record LOOP as occurring in REGION.  */

 static void
Index: gcc/sese.h
===================================================================
--- gcc/sese.h	(revision 197272)
+++ gcc/sese.h	(working copy)
@@ -275,17 +275,6 @@  new_rename_map_elt (tree old_name, tree
   return res;
 }

-/* Structure containing the mapping between the CLooG's induction
-   variable and the type of the old induction variable.  */
-typedef struct ivtype_map_elt_s
-{
-  tree type;
-  const char *cloog_iv;
-} *ivtype_map_elt;
-
-extern void debug_ivtype_map (htab_t);
-extern int eq_ivtype_map_elts (const void *, const void *);
-
 /* Free and compute again all the dominators information.  */

 static inline void