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
Paths
| Differential D16021 Authored by bdrewery on Jun 26 2018, 6:18 PM.
Details Summary This will disable the new LLVM_TARGET_ALL option which will only This only impacts the bootstrap compiler in WORLDTMP, not the target compiler Sponsored by: Dell EMC Test Plan Did buildworld with native arch, cross-arch, and testing of the universe-one-clang patch.
Diff Detail
Event TimelineHarbormaster completed remote builds in B17633: Diff 44475.Jun 26 2018, 6:18 PM2018-06-26 18:18:49 (UTC+0) bdrewery added a parent revision: D16020: Add LLVM_TARGET_ALL option..Jun 26 2018, 6:19 PM2018-06-26 18:19:08 (UTC+0) This revision is now accepted and ready to land.Jun 26 2018, 7:19 PM2018-06-26 19:19:14 (UTC+0) Comment Actions 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...) This revision now requires review to proceed.Jun 27 2018, 4:47 PM2018-06-27 16:47:09 (UTC+0) Harbormaster completed remote builds in B17670: Diff 44534.Jun 27 2018, 4:47 PM2018-06-27 16:47:10 (UTC+0) Comment Actions
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 PM2018-06-27 16:58:06 (UTC+0) Closed by commit rS335707: Clang: Only build needed target for bootstrap compiler. (authored by bdrewery). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 44475 Makefile.inc1
tools/build/options/WITH_LLVM_TARGET_ALL
|