Don't default MK_LLVM_TARGET_ALL to yes unless MK_CLANG is requested
ClosedPublic
Actions

Authored by arichardson on Jul 5 2021, 5:39 PM.

Details

Reviewers

dim
emaste

Commits

Summary

When building -DWITH_LLVM_BINUTILS -DWITHOUT_CLANG, this avoids
building a few hundred C++ source files that should not be needed
by default.

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 40314
Build 37203: arc lint + arc unit

arichardson requested review of this revision.Jul 5 2021, 5:39 PM

LGTM, but give @dim a day or so to weigh in.

This revision is now accepted and ready to land.Jul 16 2021, 2:39 AM

In D31059#702267, @emaste wrote:

LGTM, but give @dim a day or so to weigh in.

I guess we could also condition this on MK_CLANG|| MK_LLD but I'm not sure src.opts.mk is flexible enough.

This revision was automatically updated to reflect the committed changes.

Path

Size

share/

mk/

2 lines