This is an archive of the discontinued LLVM Phabricator instance.

Add a note that 3.9 is a major milestone for us.
ClosedPublic

Authored by ruiu on Aug 18 2016, 2:28 PM.

Details

Summary

Can you confirm that we are now able to link most FreeBSD userland
programs with LLD?

Diff Detail

Repository
rL LLVM

Event Timeline

ruiu updated this revision to Diff 68606.Aug 18 2016, 2:28 PM
ruiu retitled this revision from to Add a note that 3.9 is a major milestone for us..
ruiu updated this object.
ruiu added reviewers: emaste, davide.
ruiu added a subscriber: llvm-commits.

It's worth to mention that LLVM/Clang/Extra-tools/Compiler-RT/Polly/LLD (but not LLDB) could be linked with LLD on RHEL (I tried on 6) on second stage.

I really would like to see build bot with this functionality.

ruiu updated this revision to Diff 68631.Aug 18 2016, 4:09 PM
  • add ", including LLVM/Clang/LLD themselves."

It may be worth to mention LLVM CMake LLVM_ENABLE_LLD option with will add -fuse-ld=lld to CMAKE_EXE_LINKER_FLAGS CMAKE_MODULE_LINKER_FLAGS CMAKE_SHARED_LINKER_FLAGS.

ruiu updated this revision to Diff 68633.Aug 18 2016, 4:29 PM
  • Add a note about LLVM_ENABLE_LLD CMake variable.
emaste edited edge metadata.Aug 19 2016, 3:03 AM

The statement about most userland components on FreeBSD is correct.

I've been testing against FreeBSD-HEAD, which contains a few general FreeBSD cleanups for improved lld compatibility. I couldn't link the rescue tools (a sort of busybox-like multi-binary) and I also had to tweak the version script for libcxxrt (due to issues with demangled symbol matching / wildcards).

The kernel and boot loader components don't link with 3.9.

docs/ReleaseNotes.rst
29–30 ↗(On Diff #68633)

This statement seems a little confusing. I think the point is that the new functionality doesn't come at the cost of performance, compared to earlier versions of LLD which were not usable for real-world linking. But that context is not apparent here for readers of the release note, who may be learning about LLD for the first time from the note.

davide edited edge metadata.Aug 19 2016, 7:51 AM

What Ed said makes sense to me. I have no strong opinion about the 2x faster than gold fact. If you want to back it up with real numbers, you can use the measurements I and Rafael got for our talk @BSDCan.

ruiu updated this revision to Diff 68689.Aug 19 2016, 8:07 AM
ruiu edited edge metadata.
  • removed a mention about speed
This revision was automatically updated to reflect the committed changes.