This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Remove some dead code in llvm_install_library_symlink()
ClosedPublic

Authored by tstellar on Jan 28 2021, 11:00 PM.

Diff Detail

Event Timeline

tstellar created this revision.Jan 28 2021, 11:00 PM
tstellar requested review of this revision.Jan 28 2021, 11:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 28 2021, 11:00 PM

I'm 100% in favor of removing code, but can you explain why it's dead?

I'm 100% in favor of removing code, but can you explain why it's dead?

All calls to llvm_install_library_symlink() passed the ALWAYS_GENERATE argument, so the condition was always false.

smeenai accepted this revision.Feb 1 2021, 9:29 AM

LGTM

I'm also confused how the ALWAYS_GENERATE name related to its actual functionality ... might have been something historic?

This revision is now accepted and ready to land.Feb 1 2021, 9:29 AM
This revision was landed with ongoing or failed builds.Feb 10 2021, 10:13 AM
This revision was automatically updated to reflect the committed changes.