[PATCH] D69740: [profile] Support counter relocation at runtime
Petr Hosek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 1 16:25:57 PDT 2019
phosek added a comment.
This is an alternative to D69700 <https://reviews.llvm.org/D69700>. I'm still working on tests but wanted to share this to get some early feedback. One open question is how to control when to enable the continuous profile update. Currently it's always enabled when `-mllvm -runtime-counter-relocation=true` was using during compilation, but the runtime behavior is technically orthogonal (i.e. you can decide to compile with relocatable counters but don't use the online profile sync at runtime) so maybe there should be a separate mechanism. The continuous mode that landed in D68351 <https://reviews.llvm.org/D68351> uses the `%c` filename pattern, but don't support filename patterns nor environment variables on Fuchsia; we could use `-mllvm -runtime-counter-relocation=true` + `%c` on other platforms, whereas on Fuchsia this mode would be always used by default.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69740/new/
https://reviews.llvm.org/D69740
More information about the llvm-commits
mailing list