| Message ID | CAGqM8fYWy-z7oesChP+eOr1EEs7nsBcGPcr2c4r5v4edbDLbyQ@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
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
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.
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