Page MenuHomeFreeBSD
Differential D31059

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.
Tags
None
Referenced Files
F170602249: D31059.id91812.diff
Sat, Sep 5, 2:35 PM
F170600659: D31059.id.diff
Sat, Sep 5, 2:25 PM
Unknown Object (File)
Thu, Sep 3, 11:54 AM
Unknown Object (File)
Thu, Sep 3, 7:38 AM
Unknown Object (File)
Wed, Sep 2, 6:02 PM
Unknown Object (File)
Wed, Sep 2, 6:16 AM
Unknown Object (File)
Tue, Sep 1, 12:10 PM
Unknown Object (File)
Tue, Sep 1, 3:23 AM
View All Files
Subscribers
bdrewery
imp

Details

Reviewers
dim
emaste
Commits
rGe7e22476d1d7: Don't default MK_LLVM_TARGET_ALL to yes unless MK_CLANG is requested
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

arichardson created this revision.Jul 5 2021, 5:39 PM
Herald added subscribers: imp, bdrewery. · View Herald TranscriptJul 5 2021, 5:39 PM
arichardson requested review of this revision.Jul 5 2021, 5:39 PM
Harbormaster completed remote builds in B40314: Diff 91812.Jul 5 2021, 5:39 PM
emaste accepted this revision.Jul 16 2021, 2:39 AM

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
arichardson added a comment.Jul 19 2021, 2:02 PM
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.

Closed by commit rGe7e22476d1d7: Don't default MK_LLVM_TARGET_ALL to yes unless MK_CLANG is requested (authored by arichardson). · Explain WhySep 6 2021, 9:25 AM
This revision was automatically updated to reflect the committed changes.
arichardson added a commit: rGe7e22476d1d7: Don't default MK_LLVM_TARGET_ALL to yes unless MK_CLANG is requested.

Revision Contents
Changeset List

PathSize
share/
mk/
2 lines

Diff 94712

View Options

share/mk/src.opts.mk

Loading...