[PATCH] D116976: [CMake] Passthrough OSX CMake options to builtins and runtimes

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 13 18:37:23 PST 2022


smeenai added a comment.

In D116976#3242464 <https://reviews.llvm.org/D116976#3242464>, @phosek wrote:

> In D116976#3242390 <https://reviews.llvm.org/D116976#3242390>, @smeenai wrote:
>
>> Actually, could you give an example of the type of variables you want to pass in? I thought the CMAKE_OSX_* options were mostly for setting OS and architecture, which the Darwin compiler-rt build does itself anyway.
>
> Yes, the concrete example we ran into recently is `CMAKE_OSX_DEPLOYMENT_TARGET` which is set here https://github.com/llvm/llvm-project/blob/58ec17cb4ea4f3b76f0c948cf4c894ea7ea43552/clang/cmake/caches/Fuchsia-stage2.cmake#L49. In the future I'd also like to take advantage of `CMAKE_OSX_ARCHITECTURES` to build libc++ as a universal library.
>
> I'm not particular happy with this approach since the runtimes build might accidentally pick up a flag you didn't intend to, which is the motivation for D117263 <https://reviews.llvm.org/D117263>.

Ah, right, I forgot that runtimes other than compiler-rt don't do any of its special OS/architecture handling. This makes sense then.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116976



More information about the llvm-commits mailing list