[PATCH] [Profile] compiler-rt support for setting profile output from command line
David
davidxl at google.com
Tue Apr 28 09:39:07 PDT 2015
================
Comment at: lib/profile/InstrProfiling.h:90
@@ +89,3 @@
+ */
+void __llvm_profile_set_filename_env_override(const char *Name);
+
----------------
Is it better to merge this interface with the previous one but with an additional parameter ?
================
Comment at: lib/profile/InstrProfilingFile.c:187
@@ -169,2 +186,3 @@
__attribute__((visibility("hidden")))
void __llvm_profile_set_filename(const char *Filename) {
+ setFilenamePossiblyWithPid(Filename);
----------------
The behavior of this interface changes (improves) slightly. Is this intended?
http://reviews.llvm.org/D9323
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list