Page MenuHomeFreeBSD

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

Authored by arichardson on Jul 5 2021, 5:39 PM.
Tags
None
Referenced Files
F132429755: D31059.id94712.diff
Thu, Oct 16, 9:09 PM
F132429753: D31059.id.diff
Thu, Oct 16, 9:09 PM
F132429748: D31059.id91812.diff
Thu, Oct 16, 9:09 PM
F132389035: D31059.diff
Thu, Oct 16, 12:04 PM
Unknown Object (File)
Tue, Oct 14, 1:35 AM
Unknown Object (File)
Thu, Oct 9, 7:20 PM
Unknown Object (File)
Wed, Oct 8, 9:18 AM
Unknown Object (File)
Sat, Oct 4, 6:19 AM
Subscribers

Details

Summary

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

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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

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.