public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "tejohnson at google dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug gcov-profile/54487] [4.8 Regression] profiledbootstrap broken by r190952
Date: Tue, 11 Sep 2012 18:57:00 -0000	[thread overview]
Message-ID: <bug-54487-4-rlOxHGXWpo@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-54487-4@http.gcc.gnu.org/bugzilla/>

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54487

--- Comment #24 from Teresa Johnson <tejohnson at google dot com> 2012-09-11 18:57:05 UTC ---
On Tue, Sep 11, 2012 at 11:14 AM, markus at trippelsdorf dot de
<gcc-bugzilla@gcc.gnu.org> wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54487
>
> --- Comment #23 from Markus Trippelsdorf <markus at trippelsdorf dot de> 2012-09-11 18:14:52 UTC ---
> gcc/gcov-io.h has:
> #if defined (HOST_HAS_F_SETLKW)
> #define GCOV_LOCKED 1
> #else
> #define GCOV_LOCKED 0
> #endif
>
> But HOST_HAS_F_SETLKW isn't defined anywhere else AFAICS:
> gcc % git grep HOST_HAS_F_SETLKW
> gcc/gcov-io.h:#if defined (HOST_HAS_F_SETLKW)
> gcc %

Maybe it is as simple as that?! I thought I saw that GCOV_LOCKED was
set for my compile, but that may have been on the libgcov compile.

In fact, just above the code Markus shows from gcov-io.h, when
IN_LIBGCOV, GCOV_LOCKED is set based on TARGET_POSIX_IO:

#if defined (TARGET_POSIX_IO)
#define GCOV_LOCKED 1
#else
#define GCOV_LOCKED 0
#endif

Indeed, when I look at the preprocessed libgcov.c output from its
compile command, the GCOV_LOCKED is clearly set (by looking at the
preprocessed gcov_open() code).

But when I use the compile command for coverage.c, which includes
gcov-io.c but is !IN_LIBGCOV (so GCOV_LOCKED is set based on
HOST_HAS_F_SETLKW), the preprocessed gcov_open code is that of a
!GCOV_LOCKED compile, without the call to fcntl.

So perhaps it is just the case that the libgcov code is that writes
the gcda files is doing the locking, but the read on profile-use is
not!

Anyone know how HOST_HAS_F_SETLKW was supposed to be set? I do see
that my configure is setting HAVE_FCNTL_H, perhaps that was intended?

Teresa

>
> --
> Configure bugmail: http://gcc.gnu.org/bugzilla/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.


  parent reply	other threads:[~2012-09-11 18:57 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-05  7:11 [Bug gcov-profile/54487] New: " markus at trippelsdorf dot de
2012-09-05  7:27 ` [Bug gcov-profile/54487] " pinskia at gcc dot gnu.org
2012-09-05  9:18 ` rguenth at gcc dot gnu.org
2012-09-05 12:41 ` hjl.tools at gmail dot com
2012-09-05 15:41 ` hjl.tools at gmail dot com
2012-09-05 15:46 ` markus at trippelsdorf dot de
2012-09-05 16:17 ` hjl.tools at gmail dot com
2012-09-05 19:03 ` tejohnson at google dot com
2012-09-05 19:20 ` markus at trippelsdorf dot de
2012-09-06 18:59 ` tejohnson at google dot com
2012-09-06 19:35 ` markus at trippelsdorf dot de
2012-09-06 20:03 ` tejohnson at google dot com
2012-09-06 20:07 ` hjl.tools at gmail dot com
2012-09-06 20:24 ` tejohnson at google dot com
2012-09-06 20:49 ` hjl.tools at gmail dot com
2012-09-07  5:19 ` tejohnson at google dot com
2012-09-07 13:50 ` tejohnson at gcc dot gnu.org
2012-09-11 17:25 ` tejohnson at google dot com
2012-09-11 17:29 ` hjl.tools at gmail dot com
2012-09-11 17:39 ` tejohnson at google dot com
2012-09-11 17:45 ` davidxl at google dot com
2012-09-11 18:05 ` tejohnson at google dot com
2012-09-11 18:08 ` davidxl at google dot com
2012-09-11 18:11 ` hjl.tools at gmail dot com
2012-09-11 18:15 ` markus at trippelsdorf dot de
2012-09-11 18:57 ` tejohnson at google dot com [this message]
2012-09-11 18:59 ` jakub at gcc dot gnu.org
2012-09-11 19:05 ` jakub at gcc dot gnu.org
2012-09-11 19:08 ` tejohnson at google dot com
2012-09-13  4:59 ` tejohnson at gcc dot gnu.org
2012-09-13  6:39 ` markus at trippelsdorf dot de
2012-09-13 13:33 ` tejohnson at gcc dot gnu.org
2012-09-13 16:17 ` hubicka at gcc dot gnu.org
2012-09-14 21:07 ` tejohnson at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-54487-4-rlOxHGXWpo@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).