authorgravatar for kubkon@jakubkonka.comJakub Konka <kubkon@jakubkonka.com> 2023-04-01 22:44:05+02:00
committergravatar for kubkon@jakubkonka.comJakub Konka <kubkon@jakubkonka.com> 2023-04-02 10:18:43+02:00
log43a6384e9c6ee6f3e250172f6f655782e808afed
tree6525daf2717615fb9fe06d781a1d598039b2b66e
parentab44b454d0408a7968354f8889a2e57c02396153

link-test: adjust test/link/bugs/macho/13056 to latest changes on macOS 13.3

Latest macOS 13.3 rolled out LLVM 15 and thus the way `nullptr_t` is defined within the `libc++`: https://github.com/llvm/llvm-project/commit/157bbe6aea22e87c822f6cda3cd404b8f657dce4 This seems to require including `/usr/include` with `-isystem` directive rather than `-I`. Otherwise we get clang miscompilation issues due to missing `nullptr_t` declaration.

1 files changed, 1 insertions(+), 1 deletions(-)

test/link/macho/bugs/13056/build.zig+1-1
...@@ -23,7 +23,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize...@@ -23,7 +23,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize
23 .name = "test",23 .name = "test",
24 .optimize = optimize,24 .optimize = optimize,
25 });25 });
26 exe.addIncludePath(std.fs.path.join(b.allocator, &.{ sdk.path, "/usr/include" }) catch unreachable);26 exe.addSystemIncludePath(std.fs.path.join(b.allocator, &.{ sdk.path, "/usr/include" }) catch unreachable);
27 exe.addIncludePath(std.fs.path.join(b.allocator, &.{ sdk.path, "/usr/include/c++/v1" }) catch unreachable);27 exe.addIncludePath(std.fs.path.join(b.allocator, &.{ sdk.path, "/usr/include/c++/v1" }) catch unreachable);
28 exe.addCSourceFile("test.cpp", &.{28 exe.addCSourceFile("test.cpp", &.{
29 "-nostdlib++",29 "-nostdlib++",