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: [ping patch] enhancing gengtype for plugins?


Diego Novillo wrote:
2009/6/12 Basile STARYNKEVITCH <basile@starynkevitch.net>:

Index: gcc/doc/gty.texi
===================================================================
--- gcc/doc/gty.texi	(revision 148436)
+++ gcc/doc/gty.texi	(working copy)
@@ -450,6 +450,13 @@ For language frontends, there is another file that
 somewhere.  It will be called @file{gtype-@var{lang}.h}, where
 @var{lang} is the name of the subdirectory the language is contained in.

+Plugins can add additional root tables. Run the @code{gengtype}
+utility in plugin mode as @code{gengtype -p @var{source-dir}
+@var{file-list} @var{plugin*.c}} with your plugin files
+@var{plugin*.c} using @code{GTY} to generate the corresponding
+@var{gt-plugin*.h} files.
+
+

Add a note stating that for this to work, a build tree for the
GCC version to be used with the plugin must be present.
I did add the note you suggested. I am not entirely sure a build tree is always needed. Perhaps one could hack a suitable file-list ... This is some experiment I have to do..


+ "\t| gengtype -p srcdir input-list plugin... #for plugins");

Change to
"\t gengtype [-p] srcdir input-list [file1 file2 ... fileN]");
Diego, you might have not read (or at least you forgot to paste) the previous line.
I commited exactly:
else
fatal ("usage: gengtype [-p] srcdir input-list [file1 file2 ... fileN]");
I hope it is what you meant.



OK with those changes.

Committed to trunk as 148515 after having bootstrapped it on x86-64-linux-gnu (ie Debian/Sid/AMD64)



A very big and warm thanks for the review.


Regards.

--
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 mines, sont seulement les miennes} ***


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