[TSan] Add CMake check for minimal SDK requirements on Darwin

[yln]

Summary

Fails with the following message in the error case:

CMake Error at /path/to/llvm-project/compiler-rt/lib/tsan/CMakeLists.txt:119 (message):
  Building the TSan runtime requires at least macOS SDK 10.12

Fixes #44682.
https://bugs.llvm.org/show_bug.cgi?id=44682

[yln]

@dmajor: are you okay with this?

[dmajor]

In D74501#1879277, @yln wrote:

@dmajor: are you okay with this?

This patch is fine from my side. I was hoping that someone more involved with this code would do the formal signoff, but it seems nobody is speaking up...

[delcypher]

compiler-rt/cmake/Modules/CompilerRTDarwinUtils.cmake

128 ↗

(On Diff #244256)

@yln Why are you changing this? This is a different piece of functionality and this verbose-ness is also deliberate. Older versions of CMake do not support VERSION_GREATER_EQUAL and so this change will break the build for older CMake versions.

compiler-rt/lib/tsan/CMakeLists.txt

117

You need to guard this check on osx being present in ${TSAN_SUPPORTED_OS}. It's possible to have compiler-rt not target osx so we shouldn't fatal error when we aren't actually building for macOS.

[yln]

compiler-rt/cmake/Modules/CompilerRTDarwinUtils.cmake

128 ↗

(On Diff #244256)

Ahh, I didn't realize VERSION_GREATER_EQUAL isn't available in older versions of CMake. Good catch!

compiler-rt/lib/tsan/CMakeLists.txt

117

I would like to fail the build for "macOS SDK < 10.12 or aligned SDK versions on other platforms". Do you know how to best express that?

[yln]

Add comment explaining that we use macOS to check aligned SDK versions. Improve error message.

[delcypher]

LGTM.

compiler-rt/lib/tsan/CMakeLists.txt

set(TSAN_HEADERS
   rtl/tsan_sync.h
   rtl/tsan_trace.h
   rtl/tsan_update_shadow_word_inl.h)
 
 set(TSAN_RUNTIME_LIBRARIES)
 add_compiler_rt_component(tsan)
 
 if(APPLE)
+  # Ideally we would check the SDK version for the actual platform we are
+  # building for here.  To make our lifes easier we assume the host SDK setup is
+  # sane and use the macOS SDK version as a proxy for aligned SDKs.
+  find_darwin_sdk_version(macosx_sdk_version "macosx")
+  if ("${macosx_sdk_version}" VERSION_LESS 10.12)
+    message(FATAL_ERROR "Building the TSan runtime requires at least macOS SDK 10.12 (or aligned SDK on other platforms)")
+  endif()
+
   add_asm_sources(TSAN_ASM_SOURCES rtl/tsan_rtl_amd64.S rtl/tsan_rtl_aarch64.S)
 
   set(TSAN_LINK_LIBS ${SANITIZER_COMMON_LINK_LIBS})
 
   add_weak_symbols("ubsan" WEAK_SYMBOL_LINK_FLAGS)
   add_weak_symbols("sanitizer_common" WEAK_SYMBOL_LINK_FLAGS)
 
   add_compiler_rt_runtime(clang_rt.tsan