[profile] Fix a profile runtime build bug for darwin

[davidxl]

Summary

EXTRA_FLAGS was never passed to rt build for darwin. This means HAS_ATOMICS and HAS_FCTNL macros were not properly set for darwin runtime build -- end up with build using stub functions.

Also getpid() requires including <unistd.h> on darwin too.

[vsk]

Thanks for catching this!

compiler-rt/trunk/lib/profile/CMakeLists.txt

set(EXTRA_FLAGS
      -DCOMPILER_RT_HAS_FCNTL_LCK=1)
 endif()
 
 if(APPLE)
   add_compiler_rt_runtime(clang_rt.profile
     STATIC
     OS ${PROFILE_SUPPORTED_OS}
     ARCHS ${PROFILE_SUPPORTED_ARCH}
+    CFLAGS ${EXTRA_FLAGS}
     SOURCES ${PROFILE_SOURCES}
     PARENT_TARGET profile)
 else()
   add_compiler_rt_runtime(clang_rt.profile
     STATIC
     ARCHS ${PROFILE_SUPPORTED_ARCH}
     CFLAGS ${EXTRA_FLAGS}
     SOURCES ${PROFILE_SOURCES}
     PARENT_TARGET profile)
 endif()
 
 add_dependencies(compiler-rt profile)

compiler-rt/trunk/lib/profile/InstrProfilingUtil.c

 #include "InstrProfiling.h"
 
 #ifdef _WIN32
 #include <direct.h>
 #include <windows.h>
 #else
 #include <sys/stat.h>
 #include <sys/types.h>
-#if defined(__linux__)
 #include <unistd.h>
-#endif
 #include <fcntl.h>
 #include <errno.h>
 #endif
 
 #ifdef COMPILER_RT_HAS_UNAME
 #include <sys/utsname.h>
 #endif