Move llvm-objdump from CLANG_EXTRAS to installed by default
ClosedPublic
Actions

Authored by emaste on Dec 21 2016, 5:04 PM.

Details

Reviewers

dim
emaste

Commits

rS316423: Update clang, llvm, lld, lldb, compiler-rt and libc++ to 4.0.0 release:
rS310840: Reapply 310775, now it also builds correctly if lldb is disabled:
rS310775: Move llvm-objdump from CLANG_EXTRAS to installed by default

Summary

We currently install three tools from binutils 2.17.50: as, ld, and objdump. Work is underway to migrate to a permissively-licensed tool-chain, with one goal being the retirement of binutils 2.17.50.

LLVM's llvm-objdump is intended to be compatible with GNU objdump although it is currently missing some options and may have formatting differences. Enable it by default for testing and further investigation. It may later be changed to install as /usr/bin/objdump, it becomes a fully viable replacement.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste updated this revision to Diff 23180.Dec 21 2016, 5:04 PM

emaste retitled this revision from to Move llvm-objdump from CLANG_EXTRAS to installed by default.

emaste updated this object.

emaste edited the test plan for this revision. (Show Details)

emaste added a reviewer: dim.

emaste added subscribers: bapt, jhb, imp.

emaste added inline comments.

lib/clang/libllvm/Makefile
311–328 ↗	(On Diff #23180)	Perhaps there's something clever that can be done so that these are built for the target tools but not the host clang build? Not sure it's worth the complexity.