mirror of
https://codeberg.org/ziglang/zig.git
synced 2026-03-07 23:04:34 +01:00
build: use -ffunction-sections -fdata-sections for the Zig compiler on PowerPC
It has been observed in practice on powerpc64(le)-linux that zig.o (in various stages and build modes) is large enough that the +/- 32MB branch range of PowerPC is insufficient to reach from one end of the code section to the other. With LLVM 21, this leads to silent miscompiles that then crash at runtime. With LLVM 22, it will at least lead to branch range errors that fail the compilation, but that gets us no closer to a working compiler. By using these options, we give the linker much greater flexibility to move code and data around to satisfy these range constraints; without them, the linker is not allowed to split up the huge code and data sections of zig.o to do so. Similar issues have also been observed on powerpc-linux (32-bit), hexagon-linux, and some variations of mips(64)(el)-linux. But let's be conservative for now; those other targets can be added to the condition later. As a data point to support this change, it's worth noting that LLD started enabling these options for LTO precisely because the resulting large compilation units ran into these range issues. In some abstract sense, Zig can be seen as doing a limited form of "LTO" in the frontend, so it's not surprising that we would hit the same issues.
This commit is contained in:
parent
a996a75e06
commit
7daf0b6f46
2 changed files with 11 additions and 0 deletions
|
|
@ -787,6 +787,11 @@ else()
|
|||
set(ZIG_WASM2C_COMPILE_FLAGS "-std=c99 -O2")
|
||||
set(ZIG1_COMPILE_FLAGS "-std=c99 -Os")
|
||||
set(ZIG2_COMPILE_FLAGS "-std=c99 -O0 -fno-sanitize=undefined -fno-stack-protector")
|
||||
# Must match the condition in build.zig.
|
||||
if(ZIG_HOST_TARGET_ARCH MATCHES "^powerpc(64)?(le)?$")
|
||||
set(ZIG1_COMPILE_FLAGS "${ZIG1_COMPILE_FLAGS} -ffunction-sections -fdata-sections")
|
||||
set(ZIG2_COMPILE_FLAGS "${ZIG2_COMPILE_FLAGS} -ffunction-sections -fdata-sections")
|
||||
endif()
|
||||
if(APPLE)
|
||||
set(ZIG2_LINK_FLAGS "-Wl,-stack_size,0x10000000")
|
||||
elseif(MINGW)
|
||||
|
|
|
|||
|
|
@ -838,6 +838,12 @@ fn addCompilerStep(b: *std.Build, options: AddCompilerModOptions) *std.Build.Ste
|
|||
});
|
||||
exe.stack_size = stack_size;
|
||||
|
||||
// Must match the condition in CMakeLists.txt.
|
||||
const function_data_sections = options.target.result.cpu.arch.isPowerPC();
|
||||
|
||||
exe.link_function_sections = function_data_sections;
|
||||
exe.link_data_sections = function_data_sections;
|
||||
|
||||
return exe;
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue