ed/Odin
1
0
Fork 0
mirror of https://github.com/Ed94/Odin.git synced 2026-08-04 14:48:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #4800 from laytan/make-use-separate-modules-corrected-linkage-hidden

Browse Source 
make corrected linkage with -use-separate-modules apply hidden visibility
This commit is contained in:
gingerBill
2025-02-07 07:28:18 +00:00
committed by GitHub
parent 80d09774b4 1053ec3051
commit a9cd70a913
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/llvm_backend.cpp
+6
View File
@@ -169,11 +169,17 @@ gb_internal void lb_correct_entity_linkage(lbGenerator *gen) {
other_global = LLVMGetNamedGlobal(ec.other_module->mod, ec.cname); other_global = LLVMGetNamedGlobal(ec.other_module->mod, ec.cname);
if (other_global) { if (other_global) {
LLVMSetLinkage(other_global, LLVMWeakAnyLinkage); LLVMSetLinkage(other_global, LLVMWeakAnyLinkage);
if (!ec.e->Variable.is_export) {
LLVMSetVisibility(other_global, LLVMHiddenVisibility);
}
} }
} else if (ec.e->kind == Entity_Procedure) { } else if (ec.e->kind == Entity_Procedure) {
other_global = LLVMGetNamedFunction(ec.other_module->mod, ec.cname); other_global = LLVMGetNamedFunction(ec.other_module->mod, ec.cname);
if (other_global) { if (other_global) {
LLVMSetLinkage(other_global, LLVMWeakAnyLinkage); LLVMSetLinkage(other_global, LLVMWeakAnyLinkage);
if (!ec.e->Procedure.is_export) {
LLVMSetVisibility(other_global, LLVMHiddenVisibility);
}
} }
} }
} }