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, Jul 16, 10:23 AM
Unknown Object (File)
Mon, Jul 15, 4:43 PM
Unknown Object (File)
Sun, Jul 14, 12:18 PM
Unknown Object (File)
Jun 24 2024, 2:37 PM
Unknown Object (File)
Jun 23 2024, 11:07 AM
Unknown Object (File)
Jun 5 2024, 1:08 PM
Unknown Object (File)
Jun 4 2024, 2:30 AM
Unknown Object (File)
Jun 3 2024, 4:33 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 12107
Build 12408: 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–160

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.