As suggested in https://reviews.llvm.org/D112155.
Details
- Reviewers
- None
- Group Reviewers
Restricted Project Restricted Project Restricted Project - Commits
- rG641a141da1f2: [runtimes] Deprecate FOO_SYSROOT & friends
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
I will also cherry-pick this to release/14.x so we can move forward with D112155 in LLVM 15.
@vvereschaka @broadwaylamb clang/cmake/caches/CrossWinToARMLinux.cmake is using those variables -- we should try to use the native CMake equivalents instead.
@vvereschaka @broadwaylamb clang/cmake/caches/CrossWinToARMLinux.cmake is using those variables -- we should try to use the native CMake equivalents instead.
@ldionne , thank you. 'll update CrossWinToARMLinux.cmake accordingly.
libcxx/CMakeLists.txt | ||
---|---|---|
294 | What's the motivation for checking if CMAKE_SYSROOT is unset and not just LIBCXX_SYSROOT? |
libcxx/CMakeLists.txt | ||
---|---|---|
294 | If they are also setting CMAKE_SYSROOT, nothing will break when we remove LIBCXX_SYSROOT. I guess I could also just check for LIBCXX_SYSROOT if you have a preference for that. |
@phosek Please let me know if you care about dropping NOT CMAKE_SYSROOT AND and I can do it post-commit.
What's the motivation for checking if CMAKE_SYSROOT is unset and not just LIBCXX_SYSROOT?