Page MenuHomeFreeBSD

Add WITH_LLVM_BITCODE option.
Needs ReviewPublic

Authored by jonathan on Oct 18 2017, 2:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 14, 3:56 AM
Unknown Object (File)
Sun, Oct 12, 2:45 PM
Unknown Object (File)
Fri, Oct 10, 9:58 PM
Unknown Object (File)
Wed, Oct 8, 8:18 AM
Unknown Object (File)
Mon, Oct 6, 11:48 PM
Unknown Object (File)
Fri, Oct 3, 10:51 AM
Unknown Object (File)
Thu, Oct 2, 11:08 PM
Unknown Object (File)
Thu, Oct 2, 10:47 PM

Details

Summary

This new option controls whether to build (and install) LLVM IR versions
of binaries and libraries alongside conventional (ELF) versions. Rather
than depending implicitly on LLVM_LINK being defined, an explicit option
(that errors out if XLLVM_LINK is not provided) is clearer with a more
obvious failure mode. It also provides a clearer signal for future
changes (e.g., the ability to build ELF binaries and libraries via IR
instead of the usual object-file workflow) to depend on.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12108
Build 12409: arc lint + arc unit

Event Timeline

share/mk/bsd.lib.mk
187

I believe the current convention is to test for == "yes" when that is what you mean.

189

test and logic appear to use LLVM_LINK, so the error should refer to that.

share/mk/bsd.prog.mk
157

same comments as for bsd.lib.mk

share/mk/src.opts.mk
183

shouldn't this be in bsd.opts.mk or src.opts.mk but not both?

share/mk/bsd.lib.mk
187

share/mk/bsd.opts.mk says:

These should be tested with == "no" or != "no" in makefiles.