aboutsummaryrefslogtreecommitdiff
path: root/lib/std/debug.zig
diff options
context:
space:
mode:
authorAndrew Kelley <andrew@ziglang.org>2024-07-18 23:35:19 -0700
committerAndrew Kelley <andrew@ziglang.org>2024-07-19 00:30:32 -0700
commite8c4e79499fbb2a83a0f4fe2cac0d80e5d12a07e (patch)
treefa8018ad4597f8f51ce760c34822ad50e9a7157c /lib/std/debug.zig
parent16604a93b9159fafec3528457366ca146bf29ce5 (diff)
downloadzig-e8c4e79499fbb2a83a0f4fe2cac0d80e5d12a07e.tar.gz
zig-e8c4e79499fbb2a83a0f4fe2cac0d80e5d12a07e.zip
std.c reorganization
It is now composed of these main sections: * Declarations that are shared among all operating systems. * Declarations that have the same name, but different type signatures depending on the operating system. Often multiple operating systems share the same type signatures however. * Declarations that are specific to a single operating system. - These are imported one per line so you can see where they come from, protected by a comptime block to prevent accessing the wrong one. Closes #19352 by changing the convention to making types `void` and functions `{}`, so that it becomes possible to update `@hasDecl` sites to use `@TypeOf(f) != void` or `T != void`. Happily, this ended up removing some duplicate logic and update some bitrotted feature detection checks. A handful of types have been modified to gain namespacing and type safety. This is a breaking change. Oh, and the last usage of `usingnamespace` site is eliminated.
Diffstat (limited to 'lib/std/debug.zig')
-rw-r--r--lib/std/debug.zig50
1 files changed, 22 insertions, 28 deletions
diff --git a/lib/std/debug.zig b/lib/std/debug.zig
index 568b49f2f6..bc7023eae2 100644
--- a/lib/std/debug.zig
+++ b/lib/std/debug.zig
@@ -201,11 +201,7 @@ pub fn dumpCurrentStackTrace(start_addr: ?usize) void {
}
}
-pub const have_ucontext = @hasDecl(posix.system, "ucontext_t") and
- (native_os != .linux or switch (builtin.cpu.arch) {
- .mips, .mipsel, .mips64, .mips64el, .riscv64 => false,
- else => true,
-});
+pub const have_ucontext = posix.ucontext_t != void;
/// Platform-specific thread state. This contains register state, and on some platforms
/// information about the stack. This is not safe to trivially copy, because some platforms
@@ -237,14 +233,7 @@ pub fn relocateContext(context: *ThreadContext) void {
};
}
-pub const have_getcontext = native_os != .openbsd and native_os != .haiku and
- !builtin.target.isAndroid() and
- (native_os != .linux or switch (builtin.cpu.arch) {
- .x86,
- .x86_64,
- => true,
- else => builtin.link_libc and !builtin.target.isMusl(),
-});
+pub const have_getcontext = @TypeOf(posix.system.getcontext) != void;
/// Capture the current context. The register values in the context will reflect the
/// state after the platform `getcontext` function returns.
@@ -704,7 +693,7 @@ pub const StackIterator = struct {
}
return true;
- } else if (@hasDecl(posix.system, "msync") and native_os != .wasi and native_os != .emscripten) {
+ } else if (have_msync) {
posix.msync(aligned_memory, posix.MSF.ASYNC) catch |err| {
switch (err) {
error.UnmappedMemory => return false,
@@ -853,6 +842,11 @@ pub const StackIterator = struct {
}
};
+const have_msync = switch (native_os) {
+ .wasi, .emscripten, .windows => false,
+ else => true,
+};
+
pub fn writeCurrentStackTrace(
out_stream: anytype,
debug_info: *DebugInfo,
@@ -2078,15 +2072,15 @@ pub const DebugInfo = struct {
if (posix.dl_iterate_phdr(&ctx, error{Found}, struct {
fn callback(info: *posix.dl_phdr_info, size: usize, context: *CtxTy) !void {
_ = size;
- if (context.address < info.dlpi_addr) return;
- const phdrs = info.dlpi_phdr[0..info.dlpi_phnum];
+ if (context.address < info.addr) return;
+ const phdrs = info.phdr[0..info.phnum];
for (phdrs) |*phdr| {
if (phdr.p_type != elf.PT_LOAD) continue;
- const seg_start = info.dlpi_addr +% phdr.p_vaddr;
+ const seg_start = info.addr +% phdr.p_vaddr;
const seg_end = seg_start + phdr.p_memsz;
if (context.address >= seg_start and context.address < seg_end) {
- context.name = mem.sliceTo(info.dlpi_name, 0) orelse "";
+ context.name = mem.sliceTo(info.name, 0) orelse "";
break;
}
} else return;
@@ -2118,30 +2112,30 @@ pub const DebugInfo = struct {
fn callback(info: *posix.dl_phdr_info, size: usize, context: *CtxTy) !void {
_ = size;
// The base address is too high
- if (context.address < info.dlpi_addr)
+ if (context.address < info.addr)
return;
- const phdrs = info.dlpi_phdr[0..info.dlpi_phnum];
+ const phdrs = info.phdr[0..info.phnum];
for (phdrs) |*phdr| {
if (phdr.p_type != elf.PT_LOAD) continue;
// Overflowing addition is used to handle the case of VSDOs having a p_vaddr = 0xffffffffff700000
- const seg_start = info.dlpi_addr +% phdr.p_vaddr;
+ const seg_start = info.addr +% phdr.p_vaddr;
const seg_end = seg_start + phdr.p_memsz;
if (context.address >= seg_start and context.address < seg_end) {
// Android libc uses NULL instead of an empty string to mark the
// main program
- context.name = mem.sliceTo(info.dlpi_name, 0) orelse "";
- context.base_address = info.dlpi_addr;
+ context.name = mem.sliceTo(info.name, 0) orelse "";
+ context.base_address = info.addr;
break;
}
} else return;
- for (info.dlpi_phdr[0..info.dlpi_phnum]) |phdr| {
+ for (info.phdr[0..info.phnum]) |phdr| {
switch (phdr.p_type) {
elf.PT_NOTE => {
// Look for .note.gnu.build-id
- const note_bytes = @as([*]const u8, @ptrFromInt(info.dlpi_addr + phdr.p_vaddr))[0..phdr.p_memsz];
+ const note_bytes = @as([*]const u8, @ptrFromInt(info.addr + phdr.p_vaddr))[0..phdr.p_memsz];
const name_size = mem.readInt(u32, note_bytes[0..4], native_endian);
if (name_size != 4) continue;
const desc_size = mem.readInt(u32, note_bytes[4..8], native_endian);
@@ -2151,7 +2145,7 @@ pub const DebugInfo = struct {
context.build_id = note_bytes[16..][0..desc_size];
},
elf.PT_GNU_EH_FRAME => {
- context.gnu_eh_frame = @as([*]const u8, @ptrFromInt(info.dlpi_addr + phdr.p_vaddr))[0..phdr.p_memsz];
+ context.gnu_eh_frame = @as([*]const u8, @ptrFromInt(info.addr + phdr.p_vaddr))[0..phdr.p_memsz];
},
else => {},
}
@@ -2592,7 +2586,7 @@ pub const have_segfault_handling_support = switch (native_os) {
.windows,
=> true,
- .freebsd, .openbsd => @hasDecl(std.c, "ucontext_t"),
+ .freebsd, .openbsd => have_ucontext,
else => false,
};
@@ -2742,7 +2736,7 @@ fn handleSegfaultWindowsExtra(
label: ?[]const u8,
) noreturn {
const exception_address = @intFromPtr(info.ExceptionRecord.ExceptionAddress);
- if (@hasDecl(windows, "CONTEXT")) {
+ if (windows.CONTEXT != void) {
nosuspend switch (panic_stage) {
0 => {
panic_stage = 1;