This is an archive of the discontinued LLVM Phabricator instance.

[gn build] Add all non-test build files for lld
ClosedPublic

Authored by thakis on Dec 11 2018, 7:37 PM.

Details

Summary

Mostly boring. Version.inc.in processing has a potentially interesting part which I've punted on for now (LLD_REVISION and LLD_REPOSITORY are set to empty strings for now).

lld now builds in the gn build. But no symlinks to it are created yet, so it can't be meaningfully run yet.

Diff Detail

Repository
rL LLVM

Event Timeline

thakis created this revision.Dec 11 2018, 7:37 PM
thakis edited the summary of this revision. (Show Details)Dec 12 2018, 7:52 AM
phosek accepted this revision.Dec 12 2018, 9:36 AM

LGTM

llvm/utils/gn/secondary/lld/COFF/BUILD.gn
10 ↗(On Diff #177816)

Now that I'm looking at this, it feels a bit awkward for lld config to be in //llvm/... rather than something like //lld:lld_config.

This revision is now accepted and ready to land.Dec 12 2018, 9:36 AM
thakis marked an inline comment as done.Dec 12 2018, 9:59 AM

Thanks!

llvm/utils/gn/secondary/lld/COFF/BUILD.gn
10 ↗(On Diff #177816)

Happy to move it in a follow-up, just say the word :-)

This revision was automatically updated to reflect the committed changes.