Re: [PATCH] Add an intermediate coverage format for gcov

2011-10-04 Thread शरद सिंघई
Sorry about the garbled message. My mistake with the mailer. Here is what I really intended to send. This patch was earlier submitted to google/main, but I propose itfor trunk as well. This patch adds an intermediate coverage format (enabled via 'gcov-i'). This is a compact format as it does

[PATCH] Add an intermediate coverage format for gcov

2011-10-04 Thread शरद सिंघई
This patch was earlier submitted to google/main, but I propose itfor the trunk as well. This patch adds an intermediate coverage format (enabled via 'gcov-i'). This is a compact format as it does not require source files. The new option ('gcov -i') outputs .gcov files in an intermediate textformat

Re: [google] Add intermediate text format for gcov (issue4595053)

2011-10-01 Thread शरद सिंघई
Sorry, I didn't think about the mainline. Since you find it useful, I will propose it for inclusion in the trunk as well. Thanks, Sharad On Sat, Oct 1, 2011 at 6:23 AM, Matt Rice wrote: > 2011/6/14 Sharad Singhai (शरद सिंघई) : >> Sorry, Rietveld didn't send out the updated

Re: Disable early inlining while compiling for coverage (issue5173042)

2011-10-01 Thread शरद सिंघई
I have updated PR/45890 with a test case. http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45890 In any case, I am attaching a test case here as well. Sharad On Sat, Oct 1, 2011 at 10:19 AM, Xinliang David Li wrote: > > On Sat, Oct 1, 2011 at 5:17 AM, Jan Hubicka wrote: > >> Yes, this will improve

Re: Disable early inlining while compiling for coverage (issue5173042)

2011-09-30 Thread शरद सिंघई
Here is an example. In the attached file, foo.c contains only two functions, 'sum' and 'main'. The function 'sum' gets inlined into 'main' (with -O2). gcc --coverage -O2 foo.c ./a.out gcov -b foo.c Now the coverage data for the 'if' condition in 'sum' looks like this: (in attached file foo.c.gcov

Re: [google] fix kernel fdo booting issue (issue4627057)

2011-06-22 Thread शरद सिंघई
Okay for google/main. Sharad On Wed, Jun 22, 2011 at 11:22 PM, Rong Xu wrote: > > This change removes the define of TARGET_VTABLE_USES_DESCRIPTORS in > kernel fdo which causes kernel panic in indirect-call-profiling. > > Tested with 2.6.34 kernel. This change only affects kernel code. > > This i

Re: [google] Add intermediate text format for gcov (issue4595053)

2011-06-14 Thread शरद सिंघई
Sorry, Rietveld didn't send out the updated patch along with my mail. Here it is. Sharad 2011-06-14 Sharad Singhai Google Ref 3 * doc/gcov.texi: Document gcov intermediate format. * gcov.c (get_gcov_file_intermediate_name): New function. (output_intermedi

Re: [google] Improve locus information during if-conversion (issue4526101)

2011-06-02 Thread शरद सिंघई
This patch improves precision of the line number information during coverage mode. Yes, I need to add an example/test case. I was planning to do that before I propose this patch for trunk as well. Thanks, Sharad On Thu, Jun 2, 2011 at 4:46 AM, Diego Novillo wrote: > > On Wed, Jun 1, 2011 at 21:0

Re: [google] Use different peeling parameters with available profile (issue4438079)

2011-04-29 Thread शरद सिंघई
Thanks a bunch! Sharad On Fri, Apr 29, 2011 at 1:58 PM, wrote: > > On 2011/04/29 19:21:00, Diego Novillo wrote: >> >> On 2011/04/29 00:02:40, singhai wrote: > >> > 2011-04-28  Sharad Singhai   >> > >> >       gcc/ChangeLog.google-main >> >       * params.def: Add new p