[PATCH] D69586: [profile] Support online merging with continuous sync mode

[Vedant Kumar via Phabricator via llvm-commits]

vsk marked an inline comment as done.
vsk added inline comments.


================
Comment at: compiler-rt/lib/profile/InstrProfilingFile.c:419
+ * the open file object \p File. */
+static int writeProfileWithFileObject(const char *Filename, FILE *File) {
+  setProfileFile(File);
----------------
aganea wrote:
> These two functions are not used when targeting Windows, do you think it'll be possible to `#ifdef` them out in a follow up patch? Thanks in advance!
> ```
> [550/4755] Building C object projects\compiler-rt\lib\profile\CMakeFiles\clang_rt.profile-x86_64.dir\InstrProfilingFile.c.obj
> F:\llvm-project\compiler-rt\lib\profile\InstrProfilingFile.c(419,12): warning: unused function 'writeProfileWithFileObject' [-Wunused-function]
> static int writeProfileWithFileObject(const char *Filename, FILE *File) {
>            ^
> F:\llvm-project\compiler-rt\lib\profile\InstrProfilingFile.c(429,13): warning: unused function 'unlockProfile' [-Wunused-function]
> static void unlockProfile(int *ProfileRequiresUnlock, FILE *File) {
>             ^
> 2 warnings generated.
> ```
Should be addressed by 0d4211f4e753057feec32938e596d037d4f5a6aa.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69586/new/

https://reviews.llvm.org/D69586