This is an archive of the discontinued LLVM Phabricator instance.

[lld] Remove unused LLD_REPOSITORY
ClosedPublic

Authored by MaskRay on Aug 3 2021, 5:40 PM.

Details

Reviewers
mstorsjo
thakis
Group Reviewers
Restricted Project
Commits
rGbd484c994036: [lld] Remove unused LLD_REPOSITORY
Summary

Remnant after D72803.

Distributions who want to customize the string can customize
LLD_VERSION_STRING instead.

Diff Detail

Event Timeline

MaskRay requested review of this revision.Aug 3 2021, 5:40 PM
MaskRay created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2021, 5:40 PM
MaskRay edited the summary of this revision. (Show Details)Aug 3 2021, 5:42 PM
This revision is now accepted and ready to land.Aug 3 2021, 11:06 PM
thakis accepted this revision.Aug 4 2021, 12:14 PM

Thanks!

This revision was automatically updated to reflect the committed changes.

These variables still appear to be defined by https://github.com/llvm/llvm-project/blob/9c19b36f1c/lld/Common/CMakeLists.txt#L10-L11. Should they be removed there as well?

set(version_inc "${CMAKE_CURRENT_BINARY_DIR}/VCSVersion.inc")
set(generate_vcs_version_script "${LLVM_CMAKE_PATH}/GenerateVersionFromVCS.cmake")

These variables still appear to be defined by https://github.com/llvm/llvm-project/blob/9c19b36f1c/lld/Common/CMakeLists.txt#L10-L11. Should they be removed there as well?

set(version_inc "${CMAKE_CURRENT_BINARY_DIR}/VCSVersion.inc")
set(generate_vcs_version_script "${LLVM_CMAKE_PATH}/GenerateVersionFromVCS.cmake")

Looks like I probably should bring back the code.

The code ensures that we can get this ld.lld --version output when -DLLVM_APPEND_VC_REV=ON:

LLD 14.0.0 (git@github.com:llvm/llvm-project.git a1944386156dbbfae15dfc606a3728e08ac92cfe) (compatible with GNU linkers)