Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(73)

Issue 15662012: Fix mcount counter generation (Closed)

Created:
7 years, 6 months ago by grosse
Modified:
7 years, 6 months ago
Reviewers:
Roland McGrath
CC:
native-client-reviews_googlegroups.com, bradnelson, Nick Bray (chromium)
Base URL:
http://git.chromium.org/native_client/nacl-gcc.git@master
Visibility:
Public.

Description

Fix mcount counter generation When compiling with with -pg, the x86 newlib toolchain tries to use inline locations for the mcount counts. Additionally it leaves stray @ in the generated assembly. We should use the code path that avoids all of this by defining NO_PROFILE_COUNTERS. BUG=https://code.google.com/p/nativeclient/issues/detail?id=3492 R=mcgrathr@chromium.org Committed: https://git.chromium.org/gitweb?p=native_client/nacl-gcc.git;a=commit;h=f96a3cb

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M gcc/config/i386/nacl.h View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
grosse
7 years, 6 months ago (2013-06-12 00:21:33 UTC) #1
Roland McGrath
lgtm
7 years, 6 months ago (2013-06-12 16:35:24 UTC) #2
grosse
7 years, 6 months ago (2013-06-12 21:14:58 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as rf96a3cb (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698