Rely on macOS deployment target set by compiler-rt for sanitizers

edymtt · edymtt · commit ba2a88a33a0b · 2022-09-07T01:30:52.000-07:00
Using newer targets can lead to build failures, see llvm/llvm-project@41c5288 llvm/llvm-project@b87fc09 This has no effect currently, it is meant to support the switch to `LLVM_ENABLE_RUNTIMES` to build compiler-rt -- #58465 Addresses rdar://99140817
diff --git a/utils/build-script-impl b/utils/build-script-impl
@@ -735,7 +735,6 @@ function set_build_options_for_host() {
                 -DCOMPILER_RT_ENABLE_IOS:BOOL=FALSE
                 -DCOMPILER_RT_ENABLE_WATCHOS:BOOL=FALSE
                 -DCOMPILER_RT_ENABLE_TVOS:BOOL=FALSE
-                -DSANITIZER_MIN_OSX_VERSION="${cmake_osx_deployment_target}"
                 -DLLVM_ENABLE_MODULES:BOOL="$(true_false ${LLVM_ENABLE_MODULES})"
                 -DCMAKE_OSX_ARCHITECTURES="${architecture}"
             )

Original file line number	Diff line number	Diff line change
`@@ -735,7 +735,6 @@ function set_build_options_for_host() {`
`735`	`735`	`-DCOMPILER_RT_ENABLE_IOS:BOOL=FALSE`
`736`	`736`	`-DCOMPILER_RT_ENABLE_WATCHOS:BOOL=FALSE`
`737`	`737`	`-DCOMPILER_RT_ENABLE_TVOS:BOOL=FALSE`
`738`		`- -DSANITIZER_MIN_OSX_VERSION="${cmake_osx_deployment_target}"`
`739`	`738`	`-DLLVM_ENABLE_MODULES:BOOL="$(true_false ${LLVM_ENABLE_MODULES})"`
`740`	`739`	`-DCMAKE_OSX_ARCHITECTURES="${architecture}"`
`741`	`740`	`)`