motiejus/zig
1
Fork 0
Code Releases Activity

Update __chkstk_ms to have weak linkage (#20138)

Browse Source 
* Update `__chkstk_ms` to have weak linkage

`__chkstk_ms` was causing conflicts during linking in some circumstances (specifically with linking object files from Rust sources). This PR switches `__chkstk_ms` to have weak linkage.

#15107

* Update stack_probe.zig to weak linkage for all symbols
This commit is contained in:
Ryan Sepassi
2024-07-11 13:20:06 -07:00
committed by GitHub
parent 95d9292a7a
commit 45be803646
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/compiler_rt/stack_probe.zig
View File

@@ -16,16 +16,16 @@ comptime {
if (builtin.os.tag == .windows) {
// Default stack-probe functions emitted by LLVM
if (is_mingw) {
@export(_chkstk, .{ .name = "_alloca", .linkage = strong_linkage });
@export(___chkstk_ms, .{ .name = "___chkstk_ms", .linkage = strong_linkage });
@export(_chkstk, .{ .name = "_alloca", .linkage = linkage });
@export(___chkstk_ms, .{ .name = "___chkstk_ms", .linkage = linkage });
if (arch.isAARCH64()) {
@export(__chkstk, .{ .name = "__chkstk", .linkage = strong_linkage });
@export(__chkstk, .{ .name = "__chkstk", .linkage = linkage });
}
} else if (!builtin.link_libc) {
// This symbols are otherwise exported by MSVCRT.lib
@export(_chkstk, .{ .name = "_chkstk", .linkage = strong_linkage });
@export(__chkstk, .{ .name = "__chkstk", .linkage = strong_linkage });
@export(_chkstk, .{ .name = "_chkstk", .linkage = linkage });
@export(__chkstk, .{ .name = "__chkstk", .linkage = linkage });
}
}