Re: [patch] Remove unused ivtype_map symbols from sese.[hc]

2013-04-02 Thread Richard Biener 
On Mon, Apr 1, 2013 at 12:19 AM, Lawrence Crowl cr...@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  cr...@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

Re: [patch] Remove unused ivtype_map symbols from sese.[hc]

2013-04-02 Thread Lawrence Crowl 
On 4/2/13, Richard Biener richard.guent...@gmail.com wrote:
 On Mon, Apr 1, 2013 at 12:19 AM, Lawrence Crowl cr...@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.

-- 
Lawrence Crowl

[patch] Remove unused ivtype_map symbols from sese.[hc]

2013-03-31 Thread Lawrence Crowl 
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?


Index: gcc/ChangeLog

2013-03-31  Lawrence Crowl  cr...@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