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, frv-linux] Silence warning


Hi JBG,

2012-07-21  Jan-Benedict Glaw  <jbglaw@lug-owl.de>
	* config/frv/frv-protos.h: Update copyright year,
	include basic-block.h (for ce_if_block_t).
	(frv_ifcvt_machdep_init): Declare.

Approved - please apply.

My key isn't set-up for committing to GCC (only for binutils up to now.) Please commit it yourself and I'll apply for commit-after-approval rights for GCC.

Ok - I have applied a variant of your patch. I should have noted before that including other headers in *-protos.h is frowned upon. Plus in this case it is not necessary since the parameter to
frv_ifcvt_machdep_init is not used. So I just made it a void * and updated frv.c.


Cheers
  Nick

gcc/ChangeLog
2012-07-23  Jan-Benedict Glaw  <jbglaw@lug-owl.de>
	    Nick Clifton  <nickc@redhat.com>

	* config/frv/frv-protos.h (frv_ifcvt_machdep_init): Prototype.
	* config/frv/frv.c (frv_ifcvt_machdep_init): Change type of
	(unused) parameter to void *.


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