Page MenuHomeFreeBSD

Clang: Only build needed target for bootstrap compiler.
ClosedPublic

Authored by bdrewery on Jun 26 2018, 6:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 22, 7:55 AM
Unknown Object (File)
Sat, Mar 22, 5:18 AM
Unknown Object (File)
Feb 22 2025, 6:17 AM
Unknown Object (File)
Feb 21 2025, 10:23 AM
Unknown Object (File)
Feb 10 2025, 12:24 AM
Unknown Object (File)
Jan 24 2025, 7:24 PM
Unknown Object (File)
Jan 24 2025, 7:18 PM
Unknown Object (File)
Jan 18 2025, 5:39 PM
Subscribers

Details

Summary

This will disable the new LLVM_TARGET_ALL option which will only
enable the required target.

This only impacts the bootstrap compiler in WORLDTMP, not the target compiler
that will be installed.

Sponsored by: Dell EMC

Test Plan

Did buildworld with native arch, cross-arch, and testing of the universe-one-clang patch.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17633
Build 17438: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jun 26 2018, 7:19 PM

Isn't WITH_LLVM_TARGET_ALL the default, and if so, shouldn't WITHOUT_LLVM_TARGET_ALL be documented instead? (I always found these explanatory files a little confusing, though...)

Oh, I just saw the WITHOUT_ file was added in your other review.

Allow overriding MK_LLVM_TARGET_ALL (for one-clang-universe)

This revision now requires review to proceed.Jun 27 2018, 4:47 PM
In D16021#339386, @dim wrote:

Oh, I just saw the WITHOUT_ file was added in your other review.

Yeah this review is just utilizing the feature for buildworld. The other adds the option as a simpler means to enable/disable unneeded targets for users.

This revision was not accepted when it landed; it landed in state Needs Review.Jun 27 2018, 4:58 PM
This revision was automatically updated to reflect the committed changes.