diff options
| author | Jakub Konka <kubkon@jakubkonka.com> | 2024-07-04 22:03:31 +0200 |
|---|---|---|
| committer | Jakub Konka <kubkon@jakubkonka.com> | 2024-07-04 22:03:31 +0200 |
| commit | e42e12dbbf8ebae65589ff08c3b5a454d2d67441 (patch) | |
| tree | f074b0c832be0abda892d7778b44220934046f18 /src | |
| parent | 6756aaccf14e168132cabe8e0d7933f4fd51e9a9 (diff) | |
| download | zig-e42e12dbbf8ebae65589ff08c3b5a454d2d67441.tar.gz zig-e42e12dbbf8ebae65589ff08c3b5a454d2d67441.zip | |
tsan: fix wording in comments
Diffstat (limited to 'src')
| -rw-r--r-- | src/libtsan.zig | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/libtsan.zig b/src/libtsan.zig index b7d3a9dda2..bc72f9d86c 100644 --- a/src/libtsan.zig +++ b/src/libtsan.zig @@ -27,9 +27,8 @@ pub fn buildTsan(comp: *Compilation, prog_node: std.Progress.Node) BuildError!vo const target = comp.getTarget(); const root_name = switch (target.os.tag) { - // On Apple platforms, we use the same name as LLVM and Apple so that we correctly - // mark the images as instrumented when traversing them when TSAN dylib is - // initialized. + // On Apple platforms, we use the same name as LLVM because the + // TSAN library implementation hard-codes a check for these names. .macos => "clang_rt.tsan_osx_dynamic", .ios => switch (target.abi) { .simulator => "clang_rt.tsan_iossim_dynamic", @@ -290,7 +289,7 @@ pub fn buildTsan(comp: *Compilation, prog_node: std.Progress.Node) BuildError!vo try std.fmt.allocPrintZ(arena, "@rpath/{s}", .{basename}) else null; - // This is temp conditional on resolving https://github.com/llvm/llvm-project/issues/97627 upstream. + // Workaround for https://github.com/llvm/llvm-project/issues/97627 const headerpad_size: ?u32 = if (target.isDarwin()) 32 else null; const sub_compilation = Compilation.create(comp.gpa, arena, .{ .local_cache_directory = comp.global_cache_directory, |