summaryrefslogtreecommitdiffstats
path: root/compiler-rt/lib/profile/InstrProfilingRuntime.cpp
diff options
context:
space:
mode:
authorVedant Kumar <vsk@apple.com>2019-09-19 11:56:43 -0700
committerVedant Kumar <vsk@apple.com>2019-10-31 16:04:09 -0700
commitd889d1efefe9f97507e3eafa85a2e3939df9750f (patch)
tree3b0b092d7b64e5f1620b7eef3b0282d214663e95 /compiler-rt/lib/profile/InstrProfilingRuntime.cpp
parentade776b5845384bb45fcd2f7919d80f4101971a7 (diff)
downloadbcm5719-llvm-d889d1efefe9f97507e3eafa85a2e3939df9750f.tar.gz
bcm5719-llvm-d889d1efefe9f97507e3eafa85a2e3939df9750f.zip
[profile] Add a mode to continuously sync counter updates to a file
Add support for continuously syncing profile counter updates to a file. The motivation for this is that programs do not always exit cleanly. On iOS, for example, programs are usually killed via a signal from the OS. Running atexit() handlers after catching a signal is unreliable, so some method for progressively writing out profile data is necessary. The approach taken here is to mmap() the `__llvm_prf_cnts` section onto a raw profile. To do this, the linker must page-align the counter and data sections, and the runtime must ensure that counters are mapped to a page-aligned offset within a raw profile. Continuous mode is (for the moment) incompatible with the online merging mode. This limitation is lifted in https://reviews.llvm.org/D69586. Continuous mode is also (for the moment) incompatible with value profiling, as I'm not sure whether there is interest in this and the implementation may be tricky. As I have not been able to test extensively on non-Darwin platforms, only Darwin support is included for the moment. However, continuous mode may "just work" without modification on Linux and some UNIX-likes. AIUI the default value for the GNU linker's `--section-alignment` flag is set to the page size on many systems. This appears to be true for LLD as well, as its `no_nmagic` option is on by default. Continuous mode will not "just work" on Fuchsia or Windows, as it's not possible to mmap() a section on these platforms. There is a proposal to add a layer of indirection to the profile instrumentation to support these platforms. rdar://54210980 Differential Revision: https://reviews.llvm.org/D68351
Diffstat (limited to 'compiler-rt/lib/profile/InstrProfilingRuntime.cpp')
-rw-r--r--compiler-rt/lib/profile/InstrProfilingRuntime.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/compiler-rt/lib/profile/InstrProfilingRuntime.cpp b/compiler-rt/lib/profile/InstrProfilingRuntime.cpp
index 679186ef830..5dff09d7063 100644
--- a/compiler-rt/lib/profile/InstrProfilingRuntime.cpp
+++ b/compiler-rt/lib/profile/InstrProfilingRuntime.cpp
@@ -19,8 +19,9 @@ namespace {
class RegisterRuntime {
public:
RegisterRuntime() {
- __llvm_profile_register_write_file_atexit();
__llvm_profile_initialize_file();
+ if (!__llvm_profile_is_continuous_mode_enabled())
+ __llvm_profile_register_write_file_atexit();
}
};
OpenPOWER on IntegriCloud