Chromium Code Reviews Chromium Code Reviews
Issue 10697079:
Upstreaming Cygprofile for Android. (Closed)
Base URL: http://git.chromium.org/chromium/src.git@master| Index: base/android/third_party/cygprofile_startup/README.chromium |
| diff --git a/base/android/third_party/cygprofile_startup/README.chromium b/base/android/third_party/cygprofile_startup/README.chromium |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..71cb325d0e77581c023febde46e2458513f193c5 |
| --- /dev/null |
| +++ b/base/android/third_party/cygprofile_startup/README.chromium |
| @@ -0,0 +1,40 @@ |
| +Name: CygProfiler instrumentation for -finstrument-functions |
| +Short Name: CygProfiler |
| +URL: http://www.logix.cz/michal/devel/CygProfiler/ |
| +Version: 0 |
| +Date: N/A |
| +Revision: N/A |
| +License: public domain |
| +License File: cyg-profile.h |
|
Steve Block
2012/07/11 16:17:56
I don't see this file. Do you mean LICENSE?
felipeg
2012/07/11 17:10:25
Done.
|
| +Security Critical: No |
| + |
| +Description: |
| +Implementation of instrumentation functions for use with gcc's |
| +-finstrument-functions option. |
| + |
| +Local Modifications: |
| +09/07/2011: Original unmodified version of code added to third_party |
| + |
| +09/08/2011: cyg-profile.c was removed and replaced with the cyg-profile.cc file. |
| +The style has been changed to fit Google's style guide. Logging and format |
| +changed to log only first entry of a function. Logs virtual address of code |
| +section which contains profiled code, and logs timestamps of calls. The |
| +implementation was also made thread-safe with the use of stdlib sets and a |
| +pthread mutex. |
| + |
| +01/23/2012: Android: enable profiling on first __cyg_profile_func_enter() call. |
| +This is useful to profile the large number of static initializers used |
| +by Chromium on this platform. |
| + |
| +07/03/2012: |
| +Moving third_party/cygprofile/ to base/android/third_party/cygprofile/. This |
| +implementation is currently only being used by Android and is so different from |
| +the original version that it makes more sense to move to base/android/. There is |
| +yet another implementation of cygprofile in //tools/cygprofile/ which is being |
| +used by Chromium, that is also different from the open-source version, and |
| +different from the version used by Android. |
| + |
| +07/04/2012: Renaming base/android/third_party/cygprofile/ to cygprofile_startup |
| +since this version only collects profiling information from the first time a |
| +function is run. This has been used to investigate ways to improve Chrome for |
| +Android startup time. |
