This is an archive of the discontinued LLVM Phabricator instance.

build: Add LLVM_WINSYSROOT to make setting /winsysroot easy on Win
ClosedPublic

Authored by thakis on Feb 17 2021, 7:07 AM.

Details

Summary

Also add a script for sysroot management. For now, it can only create
fake sysroots that just symlink to local folders. This is useful for
testing.

Diff Detail

Event Timeline

thakis created this revision.Feb 17 2021, 7:07 AM
thakis requested review of this revision.Feb 17 2021, 7:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 17 2021, 7:07 AM
thakis updated this revision to Diff 324304.Feb 17 2021, 7:14 AM

tiny tweak: rebase_path() sysroot in gn

hans added inline comments.Feb 17 2021, 7:15 AM
llvm/utils/sysroot.py
4

Can you expand on what "manage" could mean in the future? The current filename is pretty general, so if the main purpose is to set up sysroots (fake or not) maybe a better name would be make_sysroot.py?

thakis added inline comments.Feb 17 2021, 7:16 AM
llvm/utils/sysroot.py
4

I was thinking (at least on linux): creating, uploading a created sysroot, downloading a pre-uploaded sysroot. Hence the general name :)

hans accepted this revision.Feb 17 2021, 7:24 AM
hans added inline comments.
llvm/utils/sysroot.py
4

Ah, yes that sounds pretty general. Maybe put some of that in the comment here?

This revision is now accepted and ready to land.Feb 17 2021, 7:24 AM
thakis added inline comments.Feb 17 2021, 7:25 AM
llvm/utils/sysroot.py
4

I thought I'd update the comment once more stuff works if that's ok.