[Fwd: Re: gcov in cross-compile: have a patch, seek direction]

Danny Backx danny.backx@scarlet.be
Wed May 16 20:36:00 GMT 2007


Here's the patch that I incorrectly sent to thw wrong list earlier.

As I wrote below, comments are welcome.

Proposed ChangeLog entry :
2007-05-16  Danny Backx  <danny.backx@scarlet.be>
	* doc/gcov.texi : Add description of new compiler command line
	option -fcoverage-base.
	* toplev.h (-fcoverage-base, coverage_base) : Add new command
	line option and variable.
	* coverage.c (build_gcov_info) : Add code so the new variable
	overrides the directory name used to store coverage files.
	* common.opt : Add declaration for new command line option.
	* libgcov.c (gcov_exit) : call another error handler than stderr
	when running on Windows CE.

The last entry is not really part of the new functionality. I would
rather expect a comment to either remove this or to propose this as
another patch entirely.

	Danny

-------- Forwarded Message --------
> From: Danny Backx <danny.backx@scarlet.be>
> To: Ben Elliston <bje@au1.ibm.com>
> Cc: gcc@gcc.gnu.org
> Subject: Re: gcov in cross-compile: have a patch, seek direction
> Date: Thu, 10 May 2007 21:36:49 +0200
> 
> On Fri, 2007-05-04 at 09:49 +1000, Ben Elliston wrote:
> > On Thu, 2007-05-03 at 19:55 +0200, Danny Backx wrote:
> > 
> > > Ok, a command line option is what I have. I'll try to clean up my
> > > patch shortly, and see if it still applies cleanly in a recent gcc
> > > tree. Our current version is based on gcc-4.1.0. Or is a patch against
> > > that ok ?
> > 
> > This feature would not be accepted into GCC 4.1 or 4.2, so please bring
> > your patch up to date with mainline and then submit it.  I'd be
> > surprised if it's much work to do that.
> 
> As you said, it isn't much work.
> 
> Attached is a patch with a small amount of documentation but lacking a
> ChangeLog entry. I'm sending it now already for you to comment on.
> 
> A usage example :
> dannypc: {49} make
> /home/danny/src/gcc/svn@gcc.gnu.org/build/gcc/xgcc
> -B/home/danny/src/gcc/svn@gcc.gnu.org/build/gcc --coverage -c t.c
> -fcoverage-base=/tmp  -o t1.o
> /home/danny/src/gcc/svn@gcc.gnu.org/build/gcc/xgcc
> -B/home/danny/src/gcc/svn@gcc.gnu.org/build/gcc --coverage -c t.c -o
> t2.o
> strings t1.o >t1.strings
> strings t2.o >t2.strings
> dannypc: {50} more *.strings
> ::::::::::::::
> t1.strings
> ::::::::::::::
> e304
> Hello, world !
> /tmp/t1.gcda
> ::::::::::::::
> t2.strings
> ::::::::::::::
> e304
> Hello, world !
> /home/danny/t/t2.gcda
> dannypc: {51} 
> 
> If I need to change the implementation in some way, please advise.
> 
> 	Danny
> 
-- 
Danny Backx ; danny.backx - at - scarlet.be ; http://danny.backx.info
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcov-patch
Type: text/x-patch
Size: 4880 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070516/62988744/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070516/62988744/attachment.sig>


More information about the Gcc-patches mailing list