This is an archive of the discontinued LLVM Phabricator instance.

Remove LLVM_INSTALL_CCTOOLS_SYMLINKS
ClosedPublic

Authored by thakis on Apr 23 2018, 7:54 AM.

Details

Reviewers
JDevlieghere
Summary

It used to symlink dsymutil to llvm-dsymutil, but after r327790 llvm's dsymutil binary is now called dsymutil without prefix.

r327792 then reversed the direction of the symlink if LLVM_INSTALL_CCTOOLS_SYMLINKS was set, but that looks like a buildfix and not like something anyone should need.

Diff Detail

Event Timeline

thakis created this revision.Apr 23 2018, 7:54 AM
thakis edited subscribers, added: llvm-commits; removed: cfe-commits.Apr 23 2018, 8:12 AM
JDevlieghere accepted this revision.Apr 23 2018, 8:31 AM

LGTM, Thanks!

I was under the impression that there were other consumers but it looks like that's not the case.

This revision is now accepted and ready to land.Apr 23 2018, 8:31 AM
thakis closed this revision.Apr 24 2018, 8:44 AM

r330727, thanks!