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: [PATCH] GCC symdb (gcc/) 2 of 2


On 23/04/2010 02:33, Yunfeng ZHANG wrote:
> Does gcc still work on my patch?

  Hi again, sorry for taking some time to respond, and thank you for being
persistent and patient.

  You didn't supply a ChangeLog entry yet, so here is one for the patch:

gcc/ChangeLog:

	* plugin.def (PLUGIN_CPP_TOKEN): New DEFEVENT.
	(PLUGIN_C_TOKEN): Likewise.
	* c-lex.c: Include plugin.h.
	(c_lex_with_flags): Invoke PLUGIN_CPP_TOKEN callback.
	* c-parser.c (CPP_KEYWORD): Delete from here...
	(c_id_kind): ... likewise ...
	(c_token): ... and likewise.
	(c_lex_one_token): Invoke PLUGIN_C_TOKEN callback.
	* plugin.c (register_callback): Handle PLUGIN_C_TOKEN and
	PLUGIN_CPP_TOKEN events.
	(invoke_plugin_callbacks): Likewise.
	* plugins.texi (enum plugin_event): Mention new event types.
	* c-common.h: Include c-pragma.h.
	(CPP_KEYWORD): Define here instead ...
	(c_id_kind): ... likewise ...
	(c_token): ... and likewise.

  Joseph, I don't have a fencepost account yet; can you verify that YZ's
paperwork has been processed, and if so review the original patch at:
	http://gcc.gnu.org/ml/gcc-patches/2010-01/msg00307.html

  It looks basically right, but I think it needs a patch to Makefile.in to
adjust header dependencies?  (Do we really still need to track these manually?)

    cheers,
      DaveK


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