This is an archive of the discontinued LLVM Phabricator instance.

[runtimes] Deprecate FOO_SYSROOT & friends
ClosedPublic

Authored by ldionne on Feb 15 2022, 5:19 AM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Restricted Project
Restricted Project
Commits
rG641a141da1f2: [runtimes] Deprecate FOO_SYSROOT & friends
Summary

Diff Detail

Event Timeline

ldionne created this revision.Feb 15 2022, 5:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2022, 5:19 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne requested review of this revision.Feb 15 2022, 5:19 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 15 2022, 5:19 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript

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.

ldionne updated this revision to Diff 408848.Feb 15 2022, 6:28 AM

Rebase onto main -- that should fix CI.

@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.

phosek added inline comments.Feb 15 2022, 11:24 AM
libcxx/CMakeLists.txt
294

What's the motivation for checking if CMAKE_SYSROOT is unset and not just LIBCXX_SYSROOT?

ldionne added inline comments.Feb 15 2022, 12:19 PM
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.

ldionne updated this revision to Diff 409034.Feb 15 2022, 1:17 PM

Rebase just to see CI passing.

ldionne accepted this revision as: Restricted Project, Restricted Project, Restricted Project.Feb 16 2022, 6:43 AM

@phosek Please let me know if you care about dropping NOT CMAKE_SYSROOT AND and I can do it post-commit.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 16 2022, 6:44 AM
This revision was automatically updated to reflect the committed changes.

@phosek Please let me know if you care about dropping NOT CMAKE_SYSROOT AND and I can do it post-commit.

I'm fine with the current solution.