This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: misc gengtype micro-patch (computing fn in get_output_file_for_structure)


On Mon, 8 Nov 2010 05:37:37 +0200
Laurynas Biveinis <laurynas.biveinis@gmail.com> wrote:
> 
> Thus your patch is, unfortunately, not OK, and the proper fix would be
> - replace get_output_file_for_structure with
> get_output_file_with_visibility(NULL) in
> write_splay_tree_allocator_def;
> - in get_output_file_for_structure put a gcc_assert that s is
> UNION_OR_STRUCT_P only.
> 
> Would you be willing to do this?
 
##### patch against trunk ########
Index: gcc/gengtype.c
===================================================================
--- gcc/gengtype.c	(revision 166428)
+++ gcc/gengtype.c	(working copy)
@@ -2637,9 +2637,12 @@ output_type_enum (outf_p of, type_p s)
 static outf_p
 get_output_file_for_structure (const_type_p s, type_p *param)
 {
-  const char *fn = s->u.s.line.file;
+  const char *fn;
   int i;
 
+  gcc_assert (UNION_OR_STRUCT_P (s));
+  fn = s->u.s.line.file;
+
   /* This is a hack, and not the good kind either.  */
   for (i = NUM_PARAM - 1; i >= 0; i--)
     if (param && param[i] && param[i]->kind == TYPE_POINTER
@@ -4062,7 +4065,7 @@ output_typename (outf_p of, const_type_p t)
 static void
 write_splay_tree_allocator_def (const_type_p s)
 {
-  outf_p of = get_output_file_for_structure (s, NULL);
+  outf_p of = get_output_file_with_visibility (NULL);
   oprintf (of, "void * ggc_alloc_splay_tree_");
   output_typename (of, s);
   oprintf (of, " (int sz, void * nl)\n");

########### gcc/ChangeLog entry
2010-11-08  Basile Starynkevitch  <basile@starynkevitch.net>

	* gengtype (get_output_file_for_structure): Ensure type is union 
	or struct.
	(write_splay_tree_allocator_def): Use get_output_file_with_visibility.

#####

Ok for trunk? I hope yes, since you asked it!


Cheers

-- 
Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mine, sont seulement les miennes} ***


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]