-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Open
Labels
enhancementSolving this issue will likely involve adding new logic or components to the codebase.Solving this issue will likely involve adding new logic or components to the codebase.linkingos-windows
Milestone
Description
Related:
- completely eliminate dependency on LLD #8726
- eliminate dependency on LLD for Mach-O #8727
- eliminate dependency on LLD for WebAssembly #17750
- eliminate dependency on LLD for ELF #17749
- ability to create import libs from def files without LLVM #17807
Put simply, this issue can be closed when the following diff is applied:
--- a/src/Compilation.zig
+++ b/src/Compilation.zig
@@ -899,15 +899,13 @@ pub fn create(gpa: Allocator, options: InitOptions) !*Compilation {
// Make a decision on whether to use LLD or our own linker.
const use_lld = options.use_lld orelse blk: {
- if (options.target.isDarwin()) {
- break :blk false;
- }
-
if (!build_options.have_llvm)
break :blk false;
- if (options.target.ofmt == .c)
- break :blk false;
+ switch (options.target.ofmt) {
+ .c, .macho, .coff => break :blk false,
+ else => {},
+ }
if (options.want_lto) |lto| {
if (lto) {
squeek502, lin72h and Aditya925
Metadata
Metadata
Assignees
Labels
enhancementSolving this issue will likely involve adding new logic or components to the codebase.Solving this issue will likely involve adding new logic or components to the codebase.linkingos-windows
Type
Projects
Status
Ditch LLVM