[PATCH] D55606: [gn build] Add infrastructure to create symlinks and use it to create lld's symlinks

[Nico Weber via Phabricator via llvm-commits]

thakis marked an inline comment as done.
thakis added inline comments.


================
Comment at: llvm/utils/gn/build/symlink_or_copy.py:3
+
+"""Symlinks, or on Windows copies, an existing file to a second location.
+
----------------
smeenai wrote:
> Starting with Windows 10 RS2 (version 1703, released April 2017), you can create symlinks without needing to elevate to administrator if you have developer mode configured. I would imagine that's a reasonably common configuration for LLVM devs, and the symlinks end up saving a bunch of space. Do you think it's worth considering (not necessarily as a part of this patch; it could be done later) attempting to create a symlink even on Windows, and falling back to a copy if you get a permission denied error? I've been meaning to implement something similar on the CMake side.
I think that'd be great. We probably wouldn't want this for releases (since users might have older Windows versions), but since the gn build is primarily for day-to-day development it makes a lot of sense to me.

I'd try that in a follow-up so I can test it on my Windows box if that's alright. (Or you could try making the change, it should be a < 10 line change in just this script.)


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

https://reviews.llvm.org/D55606