diff --git a/CMakeLists.txt b/CMakeLists.txt
index 96d10f1e8354..d9784090cd08 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -12,7 +12,7 @@ if(NOT CMAKE_BUILD_TYPE)
endif()
if(NOT CMAKE_INSTALL_PREFIX)
- set(CMAKE_INSTALL_PREFIX "${CMAKE_BINARY_DIR}/stage1" CACHE STRING
+ set(CMAKE_INSTALL_PREFIX "${CMAKE_BINARY_DIR}/stage2" CACHE STRING
"Directory to install zig to" FORCE)
endif()
diff --git a/build.zig b/build.zig
index 0ff528520180..c8e757dc4ea0 100644
--- a/build.zig
+++ b/build.zig
@@ -15,6 +15,7 @@ const stack_size = 32 * 1024 * 1024;
pub fn build(b: *Builder) !void {
b.setPreferredReleaseMode(.ReleaseFast);
+ const test_step = b.step("test", "Run all the tests");
const mode = b.standardReleaseOptions();
const target = b.standardTargetOptions(.{});
const single_threaded = b.option(bool, "single-threaded", "Build artifacts that run in single threaded mode");
@@ -39,8 +40,6 @@ pub fn build(b: *Builder) !void {
const docs_step = b.step("docs", "Build documentation");
docs_step.dependOn(&docgen_cmd.step);
- const toolchain_step = b.step("test-toolchain", "Run the tests for the toolchain");
-
var test_cases = b.addTest("src/test.zig");
test_cases.stack_size = stack_size;
test_cases.setBuildMode(mode);
@@ -64,10 +63,9 @@ pub fn build(b: *Builder) !void {
const only_install_lib_files = b.option(bool, "lib-files-only", "Only install library files") orelse false;
- const is_stage1 = b.option(bool, "stage1", "Build the stage1 compiler, put stage2 behind a feature flag") orelse false;
- const omit_stage2 = b.option(bool, "omit-stage2", "Do not include stage2 behind a feature flag inside stage1") orelse false;
+ const have_stage1 = b.option(bool, "enable-stage1", "Include the stage1 compiler behind a feature flag") orelse false;
const static_llvm = b.option(bool, "static-llvm", "Disable integration with system-installed LLVM, Clang, LLD, and libc++") orelse false;
- const enable_llvm = b.option(bool, "enable-llvm", "Build self-hosted compiler with LLVM backend enabled") orelse (is_stage1 or static_llvm);
+ const enable_llvm = b.option(bool, "enable-llvm", "Build self-hosted compiler with LLVM backend enabled") orelse (have_stage1 or static_llvm);
const llvm_has_m68k = b.option(
bool,
"llvm-has-m68k",
@@ -137,7 +135,7 @@ pub fn build(b: *Builder) !void {
};
const main_file: ?[]const u8 = mf: {
- if (!is_stage1) break :mf "src/main.zig";
+ if (!have_stage1) break :mf "src/main.zig";
if (use_zig0) break :mf null;
break :mf "src/stage1.zig";
};
@@ -150,7 +148,7 @@ pub fn build(b: *Builder) !void {
exe.setBuildMode(mode);
exe.setTarget(target);
if (!skip_stage2_tests) {
- toolchain_step.dependOn(&exe.step);
+ test_step.dependOn(&exe.step);
}
b.default_step.dependOn(&exe.step);
@@ -248,7 +246,7 @@ pub fn build(b: *Builder) !void {
}
};
- if (is_stage1) {
+ if (have_stage1) {
const softfloat = b.addStaticLibrary("softfloat", null);
softfloat.setBuildMode(.ReleaseFast);
softfloat.setTarget(target);
@@ -360,8 +358,7 @@ pub fn build(b: *Builder) !void {
exe_options.addOption(bool, "enable_tracy_callstack", tracy_callstack);
exe_options.addOption(bool, "enable_tracy_allocation", tracy_allocation);
exe_options.addOption(bool, "value_tracing", value_tracing);
- exe_options.addOption(bool, "is_stage1", is_stage1);
- exe_options.addOption(bool, "omit_stage2", omit_stage2);
+ exe_options.addOption(bool, "have_stage1", have_stage1);
if (tracy) |tracy_path| {
const client_cpp = fs.path.join(
b.allocator,
@@ -396,8 +393,7 @@ pub fn build(b: *Builder) !void {
test_cases_options.addOption(bool, "enable_link_snapshots", enable_link_snapshots);
test_cases_options.addOption(bool, "skip_non_native", skip_non_native);
test_cases_options.addOption(bool, "skip_stage1", skip_stage1);
- test_cases_options.addOption(bool, "is_stage1", is_stage1);
- test_cases_options.addOption(bool, "omit_stage2", omit_stage2);
+ test_cases_options.addOption(bool, "have_stage1", have_stage1);
test_cases_options.addOption(bool, "have_llvm", enable_llvm);
test_cases_options.addOption(bool, "llvm_has_m68k", llvm_has_m68k);
test_cases_options.addOption(bool, "llvm_has_csky", llvm_has_csky);
@@ -418,7 +414,7 @@ pub fn build(b: *Builder) !void {
const test_cases_step = b.step("test-cases", "Run the main compiler test cases");
test_cases_step.dependOn(&test_cases.step);
if (!skip_stage2_tests) {
- toolchain_step.dependOn(test_cases_step);
+ test_step.dependOn(test_cases_step);
}
var chosen_modes: [4]builtin.Mode = undefined;
@@ -442,11 +438,11 @@ pub fn build(b: *Builder) !void {
const modes = chosen_modes[0..chosen_mode_index];
// run stage1 `zig fmt` on this build.zig file just to make sure it works
- toolchain_step.dependOn(&fmt_build_zig.step);
+ test_step.dependOn(&fmt_build_zig.step);
const fmt_step = b.step("test-fmt", "Run zig fmt against build.zig to make sure it works");
fmt_step.dependOn(&fmt_build_zig.step);
- toolchain_step.dependOn(tests.addPkgTests(
+ test_step.dependOn(tests.addPkgTests(
b,
test_filter,
"test/behavior.zig",
@@ -457,11 +453,10 @@ pub fn build(b: *Builder) !void {
skip_non_native,
skip_libc,
skip_stage1,
- omit_stage2,
- is_stage1,
+ skip_stage2_tests,
));
- toolchain_step.dependOn(tests.addPkgTests(
+ test_step.dependOn(tests.addPkgTests(
b,
test_filter,
"lib/compiler_rt.zig",
@@ -472,11 +467,10 @@ pub fn build(b: *Builder) !void {
skip_non_native,
true, // skip_libc
skip_stage1,
- omit_stage2 or true, // TODO get these all passing
- is_stage1,
+ skip_stage2_tests or true, // TODO get these all passing
));
- toolchain_step.dependOn(tests.addPkgTests(
+ test_step.dependOn(tests.addPkgTests(
b,
test_filter,
"lib/c.zig",
@@ -487,37 +481,36 @@ pub fn build(b: *Builder) !void {
skip_non_native,
true, // skip_libc
skip_stage1,
- omit_stage2 or true, // TODO get these all passing
- is_stage1,
+ skip_stage2_tests or true, // TODO get these all passing
));
- toolchain_step.dependOn(tests.addCompareOutputTests(b, test_filter, modes));
- toolchain_step.dependOn(tests.addStandaloneTests(
+ test_step.dependOn(tests.addCompareOutputTests(b, test_filter, modes));
+ test_step.dependOn(tests.addStandaloneTests(
b,
test_filter,
modes,
skip_non_native,
enable_macos_sdk,
target,
- omit_stage2,
+ skip_stage2_tests,
b.enable_darling,
b.enable_qemu,
b.enable_rosetta,
b.enable_wasmtime,
b.enable_wine,
));
- toolchain_step.dependOn(tests.addLinkTests(b, test_filter, modes, enable_macos_sdk, omit_stage2));
- toolchain_step.dependOn(tests.addStackTraceTests(b, test_filter, modes));
- toolchain_step.dependOn(tests.addCliTests(b, test_filter, modes));
- toolchain_step.dependOn(tests.addAssembleAndLinkTests(b, test_filter, modes));
- toolchain_step.dependOn(tests.addTranslateCTests(b, test_filter));
+ test_step.dependOn(tests.addLinkTests(b, test_filter, modes, enable_macos_sdk, skip_stage2_tests));
+ test_step.dependOn(tests.addStackTraceTests(b, test_filter, modes));
+ test_step.dependOn(tests.addCliTests(b, test_filter, modes));
+ test_step.dependOn(tests.addAssembleAndLinkTests(b, test_filter, modes));
+ test_step.dependOn(tests.addTranslateCTests(b, test_filter));
if (!skip_run_translated_c) {
- toolchain_step.dependOn(tests.addRunTranslatedCTests(b, test_filter, target));
+ test_step.dependOn(tests.addRunTranslatedCTests(b, test_filter, target));
}
// tests for this feature are disabled until we have the self-hosted compiler available
- // toolchain_step.dependOn(tests.addGenHTests(b, test_filter));
+ // test_step.dependOn(tests.addGenHTests(b, test_filter));
- const std_step = tests.addPkgTests(
+ test_step.dependOn(tests.addPkgTests(
b,
test_filter,
"lib/std/std.zig",
@@ -528,14 +521,8 @@ pub fn build(b: *Builder) !void {
skip_non_native,
skip_libc,
skip_stage1,
- omit_stage2 or true, // TODO get these all passing
- is_stage1,
- );
-
- const test_step = b.step("test", "Run all the tests");
- test_step.dependOn(toolchain_step);
- test_step.dependOn(std_step);
- test_step.dependOn(docs_step);
+ true, // TODO get these all passing
+ ));
}
const exe_cflags = [_][]const u8{
diff --git a/ci/azure/build.zig b/ci/azure/build.zig
deleted file mode 100644
index 3fec5553215f..000000000000
--- a/ci/azure/build.zig
+++ /dev/null
@@ -1,976 +0,0 @@
-const std = @import("std");
-const builtin = std.builtin;
-const Builder = std.build.Builder;
-const BufMap = std.BufMap;
-const mem = std.mem;
-const ArrayList = std.ArrayList;
-const io = std.io;
-const fs = std.fs;
-const InstallDirectoryOptions = std.build.InstallDirectoryOptions;
-const assert = std.debug.assert;
-
-const zig_version = std.builtin.Version{ .major = 0, .minor = 10, .patch = 0 };
-
-pub fn build(b: *Builder) !void {
- b.setPreferredReleaseMode(.ReleaseFast);
- const mode = b.standardReleaseOptions();
- const target = b.standardTargetOptions(.{});
- const single_threaded = b.option(bool, "single-threaded", "Build artifacts that run in single threaded mode");
- const use_zig_libcxx = b.option(bool, "use-zig-libcxx", "If libc++ is needed, use zig's bundled version, don't try to integrate with the system") orelse false;
-
- const docgen_exe = b.addExecutable("docgen", "doc/docgen.zig");
- docgen_exe.single_threaded = single_threaded;
-
- const rel_zig_exe = try fs.path.relative(b.allocator, b.build_root, b.zig_exe);
- const langref_out_path = fs.path.join(
- b.allocator,
- &[_][]const u8{ b.cache_root, "langref.html" },
- ) catch unreachable;
- const docgen_cmd = docgen_exe.run();
- docgen_cmd.addArgs(&[_][]const u8{
- rel_zig_exe,
- "doc" ++ fs.path.sep_str ++ "langref.html.in",
- langref_out_path,
- });
- docgen_cmd.step.dependOn(&docgen_exe.step);
-
- const docs_step = b.step("docs", "Build documentation");
- docs_step.dependOn(&docgen_cmd.step);
-
- const is_stage1 = b.option(bool, "stage1", "Build the stage1 compiler, put stage2 behind a feature flag") orelse false;
- const omit_stage2 = b.option(bool, "omit-stage2", "Do not include stage2 behind a feature flag inside stage1") orelse false;
- const static_llvm = b.option(bool, "static-llvm", "Disable integration with system-installed LLVM, Clang, LLD, and libc++") orelse false;
- const enable_llvm = b.option(bool, "enable-llvm", "Build self-hosted compiler with LLVM backend enabled") orelse (is_stage1 or static_llvm);
- const llvm_has_m68k = b.option(
- bool,
- "llvm-has-m68k",
- "Whether LLVM has the experimental target m68k enabled",
- ) orelse false;
- const llvm_has_csky = b.option(
- bool,
- "llvm-has-csky",
- "Whether LLVM has the experimental target csky enabled",
- ) orelse false;
- const llvm_has_arc = b.option(
- bool,
- "llvm-has-arc",
- "Whether LLVM has the experimental target arc enabled",
- ) orelse false;
- const config_h_path_option = b.option([]const u8, "config_h", "Path to the generated config.h");
-
- b.installDirectory(InstallDirectoryOptions{
- .source_dir = "lib",
- .install_dir = .lib,
- .install_subdir = "zig",
- .exclude_extensions = &[_][]const u8{
- // exclude files from lib/std/compress/
- ".gz",
- ".z.0",
- ".z.9",
- "rfc1951.txt",
- "rfc1952.txt",
- // exclude files from lib/std/compress/deflate/testdata
- ".expect",
- ".expect-noinput",
- ".golden",
- ".input",
- "compress-e.txt",
- "compress-gettysburg.txt",
- "compress-pi.txt",
- "rfc1951.txt",
- // exclude files from lib/std/tz/
- ".tzif",
- // others
- "README.md",
- },
- .blank_extensions = &[_][]const u8{
- "test.zig",
- },
- });
-
- const tracy = b.option([]const u8, "tracy", "Enable Tracy integration. Supply path to Tracy source");
- const tracy_callstack = b.option(bool, "tracy-callstack", "Include callstack information with Tracy data. Does nothing if -Dtracy is not provided") orelse false;
- const tracy_allocation = b.option(bool, "tracy-allocation", "Include allocation information with Tracy data. Does nothing if -Dtracy is not provided") orelse false;
- const force_gpa = b.option(bool, "force-gpa", "Force the compiler to use GeneralPurposeAllocator") orelse false;
- const link_libc = b.option(bool, "force-link-libc", "Force self-hosted compiler to link libc") orelse enable_llvm;
- const strip = b.option(bool, "strip", "Omit debug information") orelse false;
- const value_tracing = b.option(bool, "value-tracing", "Enable extra state tracking to help troubleshoot bugs in the compiler (using the std.debug.Trace API)") orelse false;
-
- const mem_leak_frames: u32 = b.option(u32, "mem-leak-frames", "How many stack frames to print when a memory leak occurs. Tests get 2x this amount.") orelse blk: {
- if (strip) break :blk @as(u32, 0);
- if (mode != .Debug) break :blk 0;
- break :blk 4;
- };
-
- const main_file: ?[]const u8 = if (is_stage1) null else "src/main.zig";
-
- const exe = b.addExecutable("zig", main_file);
- exe.strip = strip;
- exe.install();
- exe.setBuildMode(mode);
- exe.setTarget(target);
-
- b.default_step.dependOn(&exe.step);
- exe.single_threaded = single_threaded;
-
- if (target.isWindows() and target.getAbi() == .gnu) {
- // LTO is currently broken on mingw, this can be removed when it's fixed.
- exe.want_lto = false;
- }
-
- const exe_options = b.addOptions();
- exe.addOptions("build_options", exe_options);
-
- exe_options.addOption(u32, "mem_leak_frames", mem_leak_frames);
- exe_options.addOption(bool, "skip_non_native", false);
- exe_options.addOption(bool, "have_llvm", enable_llvm);
- exe_options.addOption(bool, "llvm_has_m68k", llvm_has_m68k);
- exe_options.addOption(bool, "llvm_has_csky", llvm_has_csky);
- exe_options.addOption(bool, "llvm_has_arc", llvm_has_arc);
- exe_options.addOption(bool, "force_gpa", force_gpa);
-
- if (link_libc) {
- exe.linkLibC();
- }
-
- const is_debug = mode == .Debug;
- const enable_logging = b.option(bool, "log", "Enable debug logging with --debug-log") orelse is_debug;
- const enable_link_snapshots = b.option(bool, "link-snapshot", "Whether to enable linker state snapshots") orelse false;
-
- const opt_version_string = b.option([]const u8, "version-string", "Override Zig version string. Default is to find out with git.");
- const version = if (opt_version_string) |version| version else v: {
- const version_string = b.fmt("{d}.{d}.{d}", .{ zig_version.major, zig_version.minor, zig_version.patch });
-
- var code: u8 = undefined;
- const git_describe_untrimmed = b.execAllowFail(&[_][]const u8{
- "git", "-C", b.build_root, "describe", "--match", "*.*.*", "--tags",
- }, &code, .Ignore) catch {
- break :v version_string;
- };
- const git_describe = mem.trim(u8, git_describe_untrimmed, " \n\r");
-
- switch (mem.count(u8, git_describe, "-")) {
- 0 => {
- // Tagged release version (e.g. 0.9.0).
- if (!mem.eql(u8, git_describe, version_string)) {
- std.debug.print("Zig version '{s}' does not match Git tag '{s}'\n", .{ version_string, git_describe });
- std.process.exit(1);
- }
- break :v version_string;
- },
- 2 => {
- // Untagged development build (e.g. 0.9.0-dev.2025+ecf0050a9).
- var it = mem.split(u8, git_describe, "-");
- const tagged_ancestor = it.next() orelse unreachable;
- const commit_height = it.next() orelse unreachable;
- const commit_id = it.next() orelse unreachable;
-
- const ancestor_ver = try std.builtin.Version.parse(tagged_ancestor);
- if (zig_version.order(ancestor_ver) != .gt) {
- std.debug.print("Zig version '{}' must be greater than tagged ancestor '{}'\n", .{ zig_version, ancestor_ver });
- std.process.exit(1);
- }
-
- // Check that the commit hash is prefixed with a 'g' (a Git convention).
- if (commit_id.len < 1 or commit_id[0] != 'g') {
- std.debug.print("Unexpected `git describe` output: {s}\n", .{git_describe});
- break :v version_string;
- }
-
- // The version is reformatted in accordance with the https://semver.org specification.
- break :v b.fmt("{s}-dev.{s}+{s}", .{ version_string, commit_height, commit_id[1..] });
- },
- else => {
- std.debug.print("Unexpected `git describe` output: {s}\n", .{git_describe});
- break :v version_string;
- },
- }
- };
- exe_options.addOption([:0]const u8, "version", try b.allocator.dupeZ(u8, version));
-
- if (enable_llvm) {
- const cmake_cfg = if (static_llvm) null else findAndParseConfigH(b, config_h_path_option);
-
- if (is_stage1) {
- const softfloat = b.addStaticLibrary("softfloat", null);
- softfloat.setBuildMode(.ReleaseFast);
- softfloat.setTarget(target);
- softfloat.addIncludeDir("deps/SoftFloat-3e-prebuilt");
- softfloat.addIncludeDir("deps/SoftFloat-3e/source/8086");
- softfloat.addIncludeDir("deps/SoftFloat-3e/source/include");
- softfloat.addCSourceFiles(&softfloat_sources, &[_][]const u8{ "-std=c99", "-O3" });
- softfloat.single_threaded = single_threaded;
-
- const zig0 = b.addExecutable("zig0", null);
- zig0.addCSourceFiles(&.{"src/stage1/zig0.cpp"}, &exe_cflags);
- zig0.addIncludeDir("zig-cache/tmp"); // for config.h
- zig0.defineCMacro("ZIG_VERSION_MAJOR", b.fmt("{d}", .{zig_version.major}));
- zig0.defineCMacro("ZIG_VERSION_MINOR", b.fmt("{d}", .{zig_version.minor}));
- zig0.defineCMacro("ZIG_VERSION_PATCH", b.fmt("{d}", .{zig_version.patch}));
- zig0.defineCMacro("ZIG_VERSION_STRING", b.fmt("\"{s}\"", .{version}));
-
- for ([_]*std.build.LibExeObjStep{ zig0, exe }) |artifact| {
- artifact.addIncludeDir("src");
- artifact.addIncludeDir("deps/SoftFloat-3e/source/include");
- artifact.addIncludeDir("deps/SoftFloat-3e-prebuilt");
-
- artifact.defineCMacro("ZIG_LINK_MODE", "Static");
-
- artifact.addCSourceFiles(&stage1_sources, &exe_cflags);
- artifact.addCSourceFiles(&optimized_c_sources, &[_][]const u8{ "-std=c99", "-O3" });
-
- artifact.linkLibrary(softfloat);
- artifact.linkLibCpp();
- }
-
- try addStaticLlvmOptionsToExe(zig0);
-
- const zig1_obj_ext = target.getObjectFormat().fileExt(target.getCpuArch());
- const zig1_obj_path = b.pathJoin(&.{ "zig-cache", "tmp", b.fmt("zig1{s}", .{zig1_obj_ext}) });
- const zig1_compiler_rt_path = b.pathJoin(&.{ b.pathFromRoot("lib"), "std", "special", "compiler_rt.zig" });
-
- const zig1_obj = zig0.run();
- zig1_obj.addArgs(&.{
- "src/stage1.zig",
- "-target",
- try target.zigTriple(b.allocator),
- "-mcpu=baseline",
- "--name",
- "zig1",
- "--zig-lib-dir",
- b.pathFromRoot("lib"),
- b.fmt("-femit-bin={s}", .{b.pathFromRoot(zig1_obj_path)}),
- "-fcompiler-rt",
- "-lc",
- });
- {
- zig1_obj.addArgs(&.{ "--pkg-begin", "build_options" });
- zig1_obj.addFileSourceArg(exe_options.getSource());
- zig1_obj.addArgs(&.{ "--pkg-end", "--pkg-begin", "compiler_rt", zig1_compiler_rt_path, "--pkg-end" });
- }
- switch (mode) {
- .Debug => {},
- .ReleaseFast => {
- zig1_obj.addArg("-OReleaseFast");
- zig1_obj.addArg("--strip");
- },
- .ReleaseSafe => {
- zig1_obj.addArg("-OReleaseSafe");
- zig1_obj.addArg("--strip");
- },
- .ReleaseSmall => {
- zig1_obj.addArg("-OReleaseSmall");
- zig1_obj.addArg("--strip");
- },
- }
- if (single_threaded orelse false) {
- zig1_obj.addArg("-fsingle-threaded");
- }
-
- exe.step.dependOn(&zig1_obj.step);
- exe.addObjectFile(zig1_obj_path);
-
- // This is intentionally a dummy path. stage1.zig tries to @import("compiler_rt") in case
- // of being built by cmake. But when built by zig it's gonna get a compiler_rt so that
- // is pointless.
- exe.addPackagePath("compiler_rt", "src/empty.zig");
- }
- if (cmake_cfg) |cfg| {
- // Inside this code path, we have to coordinate with system packaged LLVM, Clang, and LLD.
- // That means we also have to rely on stage1 compiled c++ files. We parse config.h to find
- // the information passed on to us from cmake.
- if (cfg.cmake_prefix_path.len > 0) {
- b.addSearchPrefix(cfg.cmake_prefix_path);
- }
-
- try addCmakeCfgOptionsToExe(b, cfg, exe, use_zig_libcxx);
- } else {
- // Here we are -Denable-llvm but no cmake integration.
- try addStaticLlvmOptionsToExe(exe);
- }
- }
-
- const semver = try std.SemanticVersion.parse(version);
- exe_options.addOption(std.SemanticVersion, "semver", semver);
-
- exe_options.addOption(bool, "enable_logging", enable_logging);
- exe_options.addOption(bool, "enable_link_snapshots", enable_link_snapshots);
- exe_options.addOption(bool, "enable_tracy", tracy != null);
- exe_options.addOption(bool, "enable_tracy_callstack", tracy_callstack);
- exe_options.addOption(bool, "enable_tracy_allocation", tracy_allocation);
- exe_options.addOption(bool, "value_tracing", value_tracing);
- exe_options.addOption(bool, "is_stage1", is_stage1);
- exe_options.addOption(bool, "omit_stage2", omit_stage2);
- if (tracy) |tracy_path| {
- const client_cpp = fs.path.join(
- b.allocator,
- &[_][]const u8{ tracy_path, "TracyClient.cpp" },
- ) catch unreachable;
-
- // On mingw, we need to opt into windows 7+ to get some features required by tracy.
- const tracy_c_flags: []const []const u8 = if (target.isWindows() and target.getAbi() == .gnu)
- &[_][]const u8{ "-DTRACY_ENABLE=1", "-fno-sanitize=undefined", "-D_WIN32_WINNT=0x601" }
- else
- &[_][]const u8{ "-DTRACY_ENABLE=1", "-fno-sanitize=undefined" };
-
- exe.addIncludeDir(tracy_path);
- exe.addCSourceFile(client_cpp, tracy_c_flags);
- if (!enable_llvm) {
- exe.linkSystemLibraryName("c++");
- }
- exe.linkLibC();
-
- if (target.isWindows()) {
- exe.linkSystemLibrary("dbghelp");
- exe.linkSystemLibrary("ws2_32");
- }
- }
-}
-
-const exe_cflags = [_][]const u8{
- "-std=c++14",
- "-D__STDC_CONSTANT_MACROS",
- "-D__STDC_FORMAT_MACROS",
- "-D__STDC_LIMIT_MACROS",
- "-D_GNU_SOURCE",
- "-fvisibility-inlines-hidden",
- "-fno-exceptions",
- "-fno-rtti",
- "-Werror=type-limits",
- "-Wno-missing-braces",
- "-Wno-comment",
-};
-
-fn addCmakeCfgOptionsToExe(
- b: *Builder,
- cfg: CMakeConfig,
- exe: *std.build.LibExeObjStep,
- use_zig_libcxx: bool,
-) !void {
- exe.addObjectFile(fs.path.join(b.allocator, &[_][]const u8{
- cfg.cmake_binary_dir,
- "zigcpp",
- b.fmt("{s}{s}{s}", .{ exe.target.libPrefix(), "zigcpp", exe.target.staticLibSuffix() }),
- }) catch unreachable);
- assert(cfg.lld_include_dir.len != 0);
- exe.addIncludeDir(cfg.lld_include_dir);
- addCMakeLibraryList(exe, cfg.clang_libraries);
- addCMakeLibraryList(exe, cfg.lld_libraries);
- addCMakeLibraryList(exe, cfg.llvm_libraries);
-
- if (use_zig_libcxx) {
- exe.linkLibCpp();
- } else {
- const need_cpp_includes = true;
-
- // System -lc++ must be used because in this code path we are attempting to link
- // against system-provided LLVM, Clang, LLD.
- if (exe.target.getOsTag() == .linux) {
- // First we try to static link against gcc libstdc++. If that doesn't work,
- // we fall back to -lc++ and cross our fingers.
- addCxxKnownPath(b, cfg, exe, "libstdc++.a", "", need_cpp_includes) catch |err| switch (err) {
- error.RequiredLibraryNotFound => {
- exe.linkSystemLibrary("c++");
- },
- else => |e| return e,
- };
- exe.linkSystemLibrary("unwind");
- } else if (exe.target.isFreeBSD()) {
- try addCxxKnownPath(b, cfg, exe, "libc++.a", null, need_cpp_includes);
- exe.linkSystemLibrary("pthread");
- } else if (exe.target.getOsTag() == .openbsd) {
- try addCxxKnownPath(b, cfg, exe, "libc++.a", null, need_cpp_includes);
- try addCxxKnownPath(b, cfg, exe, "libc++abi.a", null, need_cpp_includes);
- } else if (exe.target.isDarwin()) {
- exe.linkSystemLibrary("c++");
- }
- }
-
- if (cfg.dia_guids_lib.len != 0) {
- exe.addObjectFile(cfg.dia_guids_lib);
- }
-}
-
-fn addStaticLlvmOptionsToExe(
- exe: *std.build.LibExeObjStep,
-) !void {
- // Adds the Zig C++ sources which both stage1 and stage2 need.
- //
- // We need this because otherwise zig_clang_cc1_main.cpp ends up pulling
- // in a dependency on llvm::cfg::Update
Loads and stores are assumed to not have side effects. If a given load or store should have side effects, such as Memory Mapped Input/Output (MMIO), use {#syntax#}volatile{#endsyntax#}. @@ -2862,19 +2863,22 @@ var foo: u8 align(4) = 100; test "global variable alignment" { try expect(@typeInfo(@TypeOf(&foo)).Pointer.alignment == 4); try expect(@TypeOf(&foo) == *align(4) u8); - const as_pointer_to_array: *[1]u8 = &foo; - const as_slice: []u8 = as_pointer_to_array; - try expect(@TypeOf(as_slice) == []align(4) u8); + const as_pointer_to_array: *align(4) [1]u8 = &foo; + const as_slice: []align(4) u8 = as_pointer_to_array; + const as_unaligned_slice: []u8 = as_slice; + try expect(as_unaligned_slice[0] == 100); } -fn derp() align(@sizeOf(usize) * 2) i32 { return 1234; } +fn derp() align(@sizeOf(usize) * 2) i32 { + return 1234; +} fn noop1() align(1) void {} fn noop4() align(4) void {} test "function alignment" { try expect(derp() == 1234); - try expect(@TypeOf(noop1) == fn() align(1) void); - try expect(@TypeOf(noop4) == fn() align(4) void); + try expect(@TypeOf(noop1) == fn () align(1) void); + try expect(@TypeOf(noop4) == fn () align(4) void); noop1(); noop4(); } @@ -3336,6 +3340,7 @@ fn doTheTest() !void { Zig allows the address to be taken of a non-byte-aligned field:
{#code_begin|test|pointer_to_non-byte_aligned_field#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -3391,7 +3396,8 @@ fn bar(x: *const u3) u3 {Pointers to non-ABI-aligned fields share the same address as the other fields within their host integer:
- {#code_begin|test|pointer_to_non-bit_aligned_field#} + {#code_begin|test|packed_struct_field_addrs#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -3407,7 +3413,7 @@ var bit_field = BitField{ .c = 3, }; -test "pointer to non-bit-aligned field" { +test "pointers of sub-byte-aligned fields share addresses" { try expect(@ptrToInt(&bit_field.a) == @ptrToInt(&bit_field.b)); try expect(@ptrToInt(&bit_field.a) == @ptrToInt(&bit_field.c)); } @@ -3438,20 +3444,22 @@ test "pointer to non-bit-aligned field" { } {#code_end#}- Packed structs have 1-byte alignment. However if you have an overaligned pointer to a packed struct, - Zig should correctly understand the alignment of fields. However there is - a bug: + Packed structs have the same alignment as their backing integer, however, overaligned + pointers to packed structs can override this:
- {#code_begin|test_err|expected type '*u32', found '*align(1) u32'#} + {#code_begin|test|overaligned_packed_struct#} +const std = @import("std"); +const expect = std.testing.expect; + const S = packed struct { a: u32, b: u32, }; test "overaligned pointer to packed struct" { - var foo: S align(4) = undefined; + var foo: S align(4) = .{ .a = 1, .b = 2 }; const ptr: *align(4) S = &foo; const ptr_to_b: *u32 = &ptr.b; - _ = ptr_to_b; + try expect(ptr_to_b.* == 2); } {#code_end#}When this bug is fixed, the above test in the documentation will unexpectedly pass, which will @@ -3698,7 +3706,7 @@ test "@tagName" {
By default, enums are not guaranteed to be compatible with the C ABI:
- {#code_begin|obj_err|parameter of type 'Foo' not allowed in function with calling convention 'C'#} + {#code_begin|obj_err|parameter of type 'test.Foo' not allowed in function with calling convention 'C'#} const Foo = enum { a, b, c }; export fn entry(foo: Foo) void { _ = foo; } {#code_end#} @@ -4004,7 +4012,7 @@ fn makeNumber() Number { This is typically used for type safety when interacting with C code that does not expose struct details. Example: - {#code_begin|test_err|expected type '*Derp', found '*Wat'#} + {#code_begin|test_err|expected type '*test.Derp', found '*test.Wat'#} const Derp = opaque {}; const Wat = opaque {}; @@ -4203,7 +4211,7 @@ test "switch on tagged union" { When a {#syntax#}switch{#endsyntax#} expression does not have an {#syntax#}else{#endsyntax#} clause, it must exhaustively list all the possible values. Failure to do so is a compile error: - {#code_begin|test_err|not handled in switch#} + {#code_begin|test_err|unhandled enumeration value#} const Color = enum { auto, off, @@ -5026,17 +5034,9 @@ test "function" { try expect(do_op(sub2, 5, 6) == -1); } {#code_end#} -Function values are like pointers:
- {#code_begin|obj#} -const assert = @import("std").debug.assert; - -comptime { - assert(@TypeOf(foo) == fn()void); - assert(@sizeOf(fn()void) == @sizeOf(?fn()void)); -} - -fn foo() void { } - {#code_end#} +There is a difference between a function body and a function pointer. + Function bodies are {#link|comptime#}-only types while function {#link|Pointers#} may be + runtime-known.
{#header_open|Pass-by-value Parameters#}Primitive types such as {#link|Integers#} and {#link|Floats#} passed as parameters @@ -6123,10 +6123,11 @@ test "float widening" { two choices about the coercion.
{#link|Zero Bit Types#} may be coerced to single-item {#link|Pointers#}, - regardless of const.
-TODO document the reasoning for this
-TODO document whether vice versa should work and why
- {#code_begin|test|coerce_zero_bit_types#} -test "coercion of zero bit types" { - var x: void = {}; - var y: *void = x; - _ = y; -} - {#code_end#} - {#header_close#} {#header_open|Type Coercion: undefined#}{#link|undefined#} can be cast to any type.
{#header_close#} @@ -6467,7 +6455,6 @@ test "peer type resolution: *const T and ?*T" {These types can only ever have one possible value, and thus @@ -6527,7 +6514,7 @@ test "turn HashMap into a set with void" {
Expressions of type {#syntax#}void{#endsyntax#} are the only ones whose value can be ignored. For example:
- {#code_begin|test_err|expression value is ignored#} + {#code_begin|test_err|ignored#} test "ignoring expression value" { foo(); } @@ -6553,37 +6540,6 @@ fn foo() i32 { } {#code_end#} {#header_close#} - - {#header_open|Pointers to Zero Bit Types#} -Pointers to zero bit types also have zero bits. They always compare equal to each other:
- {#code_begin|test|pointers_to_zero_bits#} -const std = @import("std"); -const expect = std.testing.expect; - -test "pointer to empty struct" { - const Empty = struct {}; - var a = Empty{}; - var b = Empty{}; - var ptr_a = &a; - var ptr_b = &b; - comptime try expect(ptr_a == ptr_b); -} - {#code_end#} -The type being pointed to can only ever be one value; therefore loads and stores are - never generated. {#link|ptrToInt#} and {#link|intToPtr#} are not allowed:
- {#code_begin|test_err#} -const Empty = struct {}; - -test "@ptrToInt for pointer to zero bit type" { - var a = Empty{}; - _ = @ptrToInt(&a); -} - -test "@intToPtr for pointer to zero bit type" { - _ = @intToPtr(*Empty, 0x1); -} - {#code_end#} - {#header_close#} {#header_close#} {#header_open|Result Location Semantics#} @@ -6666,7 +6622,7 @@ fn gimmeTheBiggerInteger(a: u64, b: u64) u64 {For example, if we were to introduce another function to the above snippet:
- {#code_begin|test_err|values of type 'type' must be comptime known#} + {#code_begin|test_err|unable to resolve comptime value#} fn max(comptime T: type, a: T, b: T) T { return if (a > b) a else b; } @@ -6692,7 +6648,7 @@ fn foo(condition: bool) void {For example:
- {#code_begin|test_err|operator not allowed for type 'bool'#} + {#code_begin|test_err|operator > not allowed for type 'bool'#} fn max(comptime T: type, a: T, b: T) T { return if (a > b) a else b; } @@ -6837,7 +6793,7 @@ fn performFn(start_value: i32) i32 { use a {#syntax#}comptime{#endsyntax#} expression to guarantee that the expression will be evaluated at compile-time. If this cannot be accomplished, the compiler will emit an error. For example: - {#code_begin|test_err|unable to evaluate constant expression#} + {#code_begin|test_err|comptime call of extern function#} extern fn exit() noreturn; test "foo" { @@ -6889,7 +6845,7 @@ test "fibonacci" {Imagine if we had forgotten the base case of the recursive function and tried to run the tests:
- {#code_begin|test_err|operation caused overflow#} + {#code_begin|test_err|overflow of integer type#} const expect = @import("std").testing.expect; fn fibonacci(index: u32) u32 { @@ -6913,7 +6869,8 @@ test "fibonacci" { But what would have happened if we used a signed integer? {#code_begin|test_err|evaluation exceeded 1000 backwards branches#} -const expect = @import("std").testing.expect; + {#backend_stage1#} +const assert = @import("std").debug.assert; fn fibonacci(index: i32) i32 { //if (index < 2) return index; @@ -6922,7 +6879,7 @@ fn fibonacci(index: i32) i32 { test "fibonacci" { comptime { - try expect(fibonacci(7) == 13); + try assert(fibonacci(7) == 13); } } {#code_end#} @@ -6935,8 +6892,8 @@ test "fibonacci" {What if we fix the base case, but put the wrong value in the {#syntax#}expect{#endsyntax#} line?
- {#code_begin|test_err|test "fibonacci"... FAIL (TestUnexpectedResult)#} -const expect = @import("std").testing.expect; + {#code_begin|test_err|reached unreachable#} +const assert = @import("std").debug.assert; fn fibonacci(index: i32) i32 { if (index < 2) return index; @@ -6945,16 +6902,10 @@ fn fibonacci(index: i32) i32 { test "fibonacci" { comptime { - try expect(fibonacci(7) == 99999); + try assert(fibonacci(7) == 99999); } } {#code_end#} -- What happened is Zig started interpreting the {#syntax#}expect{#endsyntax#} function with the - parameter {#syntax#}ok{#endsyntax#} set to {#syntax#}false{#endsyntax#}. When the interpreter hit - {#syntax#}@panic{#endsyntax#} it emitted a compile error because a panic during compile - causes a compile error if it is detected at compile-time. -
At container level (outside of any function), all expressions are implicitly @@ -7280,6 +7231,7 @@ pub fn main() void {
{#code_begin|exe#} {#target_linux_x86_64#} + {#backend_stage1#} pub fn main() noreturn { const msg = "hello world\n"; _ = syscall3(SYS_write, STDOUT_FILENO, @ptrToInt(msg), msg.len); @@ -7497,6 +7449,7 @@ test "global assembly" { or resumer (in the case of subsequent suspensions). {#code_begin|test|suspend_no_resume#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -7524,6 +7477,7 @@ fn func() void { {#link|@frame#} provides access to the async function frame pointer. {#code_begin|test|async_suspend_block#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -7562,6 +7516,7 @@ fn testSuspendBlock() void { never returns to its resumer and continues executing. {#code_begin|test|resume_from_suspend#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -7598,6 +7553,7 @@ fn testResumeFromSuspend(my_result: *i32) void { and the return value of the async function would be lost. {#code_begin|test|async_await#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -7642,6 +7598,7 @@ fn func() void { return value directly from the target function's frame. {#code_begin|test|async_await_sequence#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -7695,6 +7652,7 @@ fn seq(c: u8) void { {#syntax#}async{#endsyntax#}/{#syntax#}await{#endsyntax#} usage: {#code_begin|exe|async#} + {#backend_stage1#} const std = @import("std"); const Allocator = std.mem.Allocator; @@ -7773,6 +7731,7 @@ fn readFile(allocator: Allocator, filename: []const u8) ![]u8 { observe the same behavior, with one tiny difference: {#code_begin|exe|blocking#} + {#backend_stage1#} const std = @import("std"); const Allocator = std.mem.Allocator; @@ -7910,6 +7869,7 @@ comptime { {#syntax#}await{#endsyntax#} will copy the result from {#syntax#}result_ptr{#endsyntax#}. {#code_begin|test|async_struct_field_fn_pointer#} + {#backend_stage1#} const std = @import("std"); const expect = std.testing.expect; @@ -8677,6 +8637,7 @@ test "decl access by string" { allows one to, for example, heap-allocate an async function frame: {#code_begin|test|heap_allocated_frame#} + {#backend_stage1#} const std = @import("std"); test "heap allocated frame" { @@ -9423,12 +9384,6 @@ const std = @import("std"); const expect = std.testing.expect; test "vector @reduce" { - // This test regressed with LLVM 14: - // https://github.com/llvm/llvm-project/issues/55522 - // We'll skip this test unless the self-hosted compiler is being used. - // After LLVM 15 is released we can delete this line. - if (@import("builtin").zig_backend == .stage1) return; - const value = @Vector(4, i32){ 1, -1, 1, -1 }; const result = value > @splat(4, @as(i32, 0)); // result is { true, false, true, false }; @@ -9938,7 +9893,7 @@ pub fn main() void { {#header_close#} {#header_open|Index out of Bounds#}At compile-time:
- {#code_begin|test_err|index 5 outside array of size 5#} + {#code_begin|test_err|index 5 outside array of length 5#} comptime { const array: [5]u8 = "hello".*; const garbage = array[5]; @@ -9959,9 +9914,9 @@ fn foo(x: []const u8) u8 { {#header_close#} {#header_open|Cast Negative Number to Unsigned Integer#}At compile-time:
- {#code_begin|test_err|attempt to cast negative value to unsigned integer#} + {#code_begin|test_err|type 'u32' cannot represent integer value '-1'#} comptime { - const value: i32 = -1; + var value: i32 = -1; const unsigned = @intCast(u32, value); _ = unsigned; } @@ -9982,7 +9937,7 @@ pub fn main() void { {#header_close#} {#header_open|Cast Truncates Data#}At compile-time:
- {#code_begin|test_err|cast from 'u16' to 'u8' truncates bits#} + {#code_begin|test_err|type 'u8' cannot represent integer value '300'#} comptime { const spartan_count: u16 = 300; const byte = @intCast(u8, spartan_count); @@ -10017,7 +9972,7 @@ pub fn main() void {Example with addition at compile-time:
- {#code_begin|test_err|operation caused overflow#} + {#code_begin|test_err|overflow of integer type 'u8' with value '256'#} comptime { var byte: u8 = 255; byte += 1; @@ -10118,6 +10073,7 @@ test "wraparound addition and subtraction" { {#header_open|Exact Left Shift Overflow#}At compile-time:
{#code_begin|test_err|operation caused overflow#} + {#backend_stage1#} comptime { const x = @shlExact(@as(u8, 0b01010101), 2); _ = x; @@ -10137,6 +10093,7 @@ pub fn main() void { {#header_open|Exact Right Shift Overflow#}At compile-time:
{#code_begin|test_err|exact shift shifted out 1 bits#} + {#backend_stage1#} comptime { const x = @shrExact(@as(u8, 0b10101010), 2); _ = x; @@ -10200,6 +10157,7 @@ pub fn main() void { {#header_open|Exact Division Remainder#}At compile-time:
{#code_begin|test_err|exact division had a remainder#} + {#backend_stage1#} comptime { const a: u32 = 10; const b: u32 = 3; @@ -10302,7 +10260,7 @@ fn getNumberOrFail() !i32 { {#header_close#} {#header_open|Invalid Error Code#}At compile-time:
- {#code_begin|test_err|integer value 11 represents no error#} + {#code_begin|test_err|integer value '11' represents no error#} comptime { const err = error.AnError; const number = @errorToInt(err) + 10; @@ -10324,7 +10282,7 @@ pub fn main() void { {#header_close#} {#header_open|Invalid Enum Cast#}At compile-time:
- {#code_begin|test_err|has no tag matching integer value 3#} + {#code_begin|test_err|enum 'test.Foo' has no tag with value '3'#} const Foo = enum { a, b, @@ -10356,7 +10314,7 @@ pub fn main() void { {#header_open|Invalid Error Set Cast#}At compile-time:
- {#code_begin|test_err|error.B not a member of error set 'Set2'#} + {#code_begin|test_err|'error.B' not a member of error set 'error{A,C}'#} const Set1 = error{ A, B, @@ -10417,7 +10375,7 @@ fn foo(bytes: []u8) u32 { {#header_close#} {#header_open|Wrong Union Field Access#}At compile-time:
- {#code_begin|test_err|accessing union field 'float' while field 'int' is set#} + {#code_begin|test_err|access of union field 'float' while field 'int' is active#} comptime { var f = Foo{ .int = 42 }; f.float = 12.34; @@ -10509,6 +10467,7 @@ fn bar(f: *Foo) void {At compile-time:
{#code_begin|test_err|null pointer casted to type#} + {#backend_stage1#} comptime { const opt_ptr: ?*i32 = null; const ptr = @ptrCast(*i32, opt_ptr); @@ -10551,7 +10510,8 @@ const expect = std.testing.expect; test "using an allocator" { var buffer: [100]u8 = undefined; - const allocator = std.heap.FixedBufferAllocator.init(&buffer).allocator(); + var fba = std.heap.FixedBufferAllocator.init(&buffer); + const allocator = fba.allocator(); const result = try concat(allocator, "foo", "bar"); try expect(std.mem.eql(u8, "foobar", result)); } @@ -10647,7 +10607,7 @@ pub fn main() !void {String literals such as {#syntax#}"foo"{#endsyntax#} are in the global constant data section. This is why it is an error to pass a string literal to a mutable slice, like this:
- {#code_begin|test_err|cannot cast pointer to array literal to slice type '[]u8'#} + {#code_begin|test_err|expected type '[]u8', found '*const [5:0]u8'#} fn foo(s: []u8) void { _ = s; } diff --git a/lib/std/builtin.zig b/lib/std/builtin.zig index 2c2bc92c9602..69312df83853 100644 --- a/lib/std/builtin.zig +++ b/lib/std/builtin.zig @@ -866,7 +866,7 @@ pub fn panicUnwrapError(st: ?*StackTrace, err: anyerror) noreturn { pub fn panicOutOfBounds(index: usize, len: usize) noreturn { @setCold(true); - std.debug.panic("attempt to index out of bound: index {d}, len {d}", .{ index, len }); + std.debug.panic("index out of bounds: index {d}, len {d}", .{ index, len }); } pub noinline fn returnError(st: *StackTrace) void { diff --git a/lib/std/coff.zig b/lib/std/coff.zig index 7c077a9ec850..4371444b968f 100644 --- a/lib/std/coff.zig +++ b/lib/std/coff.zig @@ -383,7 +383,7 @@ const OptionalHeader = struct { image_base: u64, }; -const DebugDirectoryEntry = packed struct { +const DebugDirectoryEntry = extern struct { characteristiccs: u32, time_date_stamp: u32, major_version: u16, diff --git a/lib/std/os/linux/bpf.zig b/lib/std/os/linux/bpf.zig index 4cb9dbf5c539..147bf610d845 100644 --- a/lib/std/os/linux/bpf.zig +++ b/lib/std/os/linux/bpf.zig @@ -458,7 +458,7 @@ pub const Insn = packed struct { else ImmOrReg{ .imm = src }; - const src_type = switch (imm_or_reg) { + const src_type: u8 = switch (imm_or_reg) { .imm => K, .reg => X, }; diff --git a/lib/std/os/windows.zig b/lib/std/os/windows.zig index c79ccb511375..b0a0f6d40747 100644 --- a/lib/std/os/windows.zig +++ b/lib/std/os/windows.zig @@ -1802,7 +1802,7 @@ pub const PathSpace = struct { data: [PATH_MAX_WIDE:0]u16, len: usize, - pub fn span(self: PathSpace) [:0]const u16 { + pub fn span(self: *const PathSpace) [:0]const u16 { return self.data[0..self.len :0]; } }; diff --git a/src/Compilation.zig b/src/Compilation.zig index 060afba6948b..0914685c77f6 100644 --- a/src/Compilation.zig +++ b/src/Compilation.zig @@ -1040,24 +1040,7 @@ pub fn create(gpa: Allocator, options: InitOptions) !*Compilation { const comp = try arena.create(Compilation); const root_name = try arena.dupeZ(u8, options.root_name); - const use_stage1 = options.use_stage1 orelse blk: { - // Even though we may have no Zig code to compile (depending on `options.main_pkg`), - // we may need to use stage1 for building compiler-rt and other dependencies. - - if (build_options.omit_stage2) - break :blk true; - if (options.use_llvm) |use_llvm| { - if (!use_llvm) { - break :blk false; - } - } - - // If LLVM does not support the target, then we can't use it. - if (!target_util.hasLlvmSupport(options.target, options.target.ofmt)) - break :blk false; - - break :blk build_options.is_stage1; - }; + const use_stage1 = options.use_stage1 orelse false; const cache_mode = if (use_stage1 and !options.disable_lld_caching) CacheMode.whole @@ -1248,7 +1231,7 @@ pub fn create(gpa: Allocator, options: InitOptions) !*Compilation { break :blk lm; } else default_link_mode; - const dll_export_fns = if (options.dll_export_fns) |explicit| explicit else is_dyn_lib or options.rdynamic; + const dll_export_fns = options.dll_export_fns orelse (is_dyn_lib or options.rdynamic); const libc_dirs = try detectLibCIncludeDirs( arena, @@ -2213,8 +2196,7 @@ pub fn update(comp: *Compilation) !void { comp.c_object_work_queue.writeItemAssumeCapacity(key); } - const use_stage1 = build_options.omit_stage2 or - (build_options.is_stage1 and comp.bin_file.options.use_stage1); + const use_stage1 = build_options.have_stage1 and comp.bin_file.options.use_stage1; if (comp.bin_file.options.module) |module| { module.compile_log_text.shrinkAndFree(module.gpa, 0); module.generation += 1; @@ -2390,8 +2372,7 @@ fn flush(comp: *Compilation, prog_node: *std.Progress.Node) !void { }; comp.link_error_flags = comp.bin_file.errorFlags(); - const use_stage1 = build_options.omit_stage2 or - (build_options.is_stage1 and comp.bin_file.options.use_stage1); + const use_stage1 = build_options.have_stage1 and comp.bin_file.options.use_stage1; if (!use_stage1) { if (comp.bin_file.options.module) |module| { try link.File.C.flushEmitH(module); @@ -2849,7 +2830,7 @@ pub fn performAllTheWork( comp.work_queue_wait_group.reset(); defer comp.work_queue_wait_group.wait(); - const use_stage1 = build_options.is_stage1 and comp.bin_file.options.use_stage1; + const use_stage1 = build_options.have_stage1 and comp.bin_file.options.use_stage1; { const astgen_frame = tracy.namedFrame("astgen"); @@ -2952,9 +2933,6 @@ pub fn performAllTheWork( fn processOneJob(comp: *Compilation, job: Job) !void { switch (job) { .codegen_decl => |decl_index| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const module = comp.bin_file.options.module.?; const decl = module.declPtr(decl_index); @@ -2989,9 +2967,6 @@ fn processOneJob(comp: *Compilation, job: Job) !void { } }, .codegen_func => |func| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const named_frame = tracy.namedFrame("codegen_func"); defer named_frame.end(); @@ -3002,9 +2977,6 @@ fn processOneJob(comp: *Compilation, job: Job) !void { }; }, .emit_h_decl => |decl_index| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const module = comp.bin_file.options.module.?; const decl = module.declPtr(decl_index); @@ -3063,9 +3035,6 @@ fn processOneJob(comp: *Compilation, job: Job) !void { } }, .analyze_decl => |decl_index| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const module = comp.bin_file.options.module.?; module.ensureDeclAnalyzed(decl_index) catch |err| switch (err) { error.OutOfMemory => return error.OutOfMemory, @@ -3073,9 +3042,6 @@ fn processOneJob(comp: *Compilation, job: Job) !void { }; }, .update_embed_file => |embed_file| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const named_frame = tracy.namedFrame("update_embed_file"); defer named_frame.end(); @@ -3086,9 +3052,6 @@ fn processOneJob(comp: *Compilation, job: Job) !void { }; }, .update_line_number => |decl_index| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const named_frame = tracy.namedFrame("update_line_number"); defer named_frame.end(); @@ -3107,9 +3070,6 @@ fn processOneJob(comp: *Compilation, job: Job) !void { }; }, .analyze_pkg => |pkg| { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - const named_frame = tracy.namedFrame("analyze_pkg"); defer named_frame.end(); @@ -3455,7 +3415,7 @@ pub fn cImport(comp: *Compilation, c_src: []const u8) !CImportResult { var man = comp.obtainCObjectCacheManifest(); defer man.deinit(); - const use_stage1 = build_options.is_stage1 and comp.bin_file.options.use_stage1; + const use_stage1 = build_options.have_stage1 and comp.bin_file.options.use_stage1; man.hash.add(@as(u16, 0xb945)); // Random number to distinguish translate-c from compiling C objects man.hash.add(use_stage1); @@ -4770,7 +4730,7 @@ pub fn generateBuiltinZigSource(comp: *Compilation, allocator: Allocator) Alloca const target = comp.getTarget(); const generic_arch_name = target.cpu.arch.genericName(); - const use_stage1 = build_options.is_stage1 and comp.bin_file.options.use_stage1; + const use_stage1 = build_options.have_stage1 and comp.bin_file.options.use_stage1; const zig_backend: std.builtin.CompilerBackend = blk: { if (use_stage1) break :blk .stage1; @@ -5057,7 +5017,7 @@ fn buildOutputFromZig( .link_mode = .Static, .function_sections = true, .no_builtin = true, - .use_stage1 = build_options.is_stage1 and comp.bin_file.options.use_stage1, + .use_stage1 = build_options.have_stage1 and comp.bin_file.options.use_stage1, .want_sanitize_c = false, .want_stack_check = false, .want_stack_protector = 0, diff --git a/src/Module.zig b/src/Module.zig index 3ae61c264fea..3577115ded11 100644 --- a/src/Module.zig +++ b/src/Module.zig @@ -6529,3 +6529,7 @@ pub fn addGlobalAssembly(mod: *Module, decl_index: Decl.Index, source: []const u mod.global_assembly.putAssumeCapacityNoClobber(decl_index, duped_source); } + +pub fn wantDllExports(mod: Module) bool { + return mod.comp.bin_file.options.dll_export_fns and mod.getTarget().os.tag == .windows; +} diff --git a/src/Sema.zig b/src/Sema.zig index 091b88e7190f..aa02288df774 100644 --- a/src/Sema.zig +++ b/src/Sema.zig @@ -27358,9 +27358,6 @@ pub fn resolveTypeLayout( src: LazySrcLoc, ty: Type, ) CompileError!void { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); - switch (ty.zigTypeTag()) { .Struct => return sema.resolveStructLayout(block, src, ty), .Union => return sema.resolveUnionLayout(block, src, ty), @@ -27699,8 +27696,6 @@ fn resolveUnionFully( } pub fn resolveTypeFields(sema: *Sema, block: *Block, src: LazySrcLoc, ty: Type) CompileError!Type { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); switch (ty.tag()) { .@"struct" => { const struct_obj = ty.castTag(.@"struct").?.data; @@ -29323,8 +29318,6 @@ fn typePtrOrOptionalPtrTy( /// TODO merge these implementations together with the "advanced"/sema_kit pattern seen /// elsewhere in value.zig pub fn typeRequiresComptime(sema: *Sema, block: *Block, src: LazySrcLoc, ty: Type) CompileError!bool { - if (build_options.omit_stage2) - @panic("sadly stage2 is omitted from this build to save memory on the CI server"); return switch (ty.tag()) { .u1, .u8, diff --git a/src/codegen/llvm.zig b/src/codegen/llvm.zig index 0898c8fe87c6..d50b463606c1 100644 --- a/src/codegen/llvm.zig +++ b/src/codegen/llvm.zig @@ -1103,6 +1103,7 @@ pub const Object = struct { } llvm_global.setUnnamedAddr(.False); llvm_global.setLinkage(.External); + if (module.wantDllExports()) llvm_global.setDLLStorageClass(.Default); if (self.di_map.get(decl)) |di_node| { if (try decl.isFunction()) { const di_func = @ptrCast(*llvm.DISubprogram, di_node); @@ -1128,6 +1129,7 @@ pub const Object = struct { const exp_name = exports[0].options.name; llvm_global.setValueName2(exp_name.ptr, exp_name.len); llvm_global.setUnnamedAddr(.False); + if (module.wantDllExports()) llvm_global.setDLLStorageClass(.DLLExport); if (self.di_map.get(decl)) |di_node| { if (try decl.isFunction()) { const di_func = @ptrCast(*llvm.DISubprogram, di_node); @@ -1187,6 +1189,7 @@ pub const Object = struct { defer module.gpa.free(fqn); llvm_global.setValueName2(fqn.ptr, fqn.len); llvm_global.setLinkage(.Internal); + if (module.wantDllExports()) llvm_global.setDLLStorageClass(.Default); llvm_global.setUnnamedAddr(.True); if (decl.val.castTag(.variable)) |variable| { const single_threaded = module.comp.bin_file.options.single_threaded; diff --git a/src/codegen/llvm/bindings.zig b/src/codegen/llvm/bindings.zig index 9daa96eb8ff0..38f794cfda84 100644 --- a/src/codegen/llvm/bindings.zig +++ b/src/codegen/llvm/bindings.zig @@ -223,6 +223,9 @@ pub const Value = opaque { pub const setInitializer = LLVMSetInitializer; extern fn LLVMSetInitializer(GlobalVar: *const Value, ConstantVal: *const Value) void; + pub const setDLLStorageClass = LLVMSetDLLStorageClass; + extern fn LLVMSetDLLStorageClass(Global: *const Value, Class: DLLStorageClass) void; + pub const addCase = LLVMAddCase; extern fn LLVMAddCase(Switch: *const Value, OnVal: *const Value, Dest: *const BasicBlock) void; @@ -1482,6 +1485,12 @@ pub const CallAttr = enum(c_int) { AlwaysInline, }; +pub const DLLStorageClass = enum(c_uint) { + Default, + DLLImport, + DLLExport, +}; + pub const address_space = struct { pub const default: c_uint = 0; diff --git a/src/config.zig.in b/src/config.zig.in index a886b2d28efa..12e13815f84c 100644 --- a/src/config.zig.in +++ b/src/config.zig.in @@ -8,6 +8,5 @@ pub const enable_logging: bool = @ZIG_ENABLE_LOGGING_BOOL@; pub const enable_link_snapshots: bool = false; pub const enable_tracy = false; pub const value_tracing = false; -pub const is_stage1 = true; +pub const have_stage1 = true; pub const skip_non_native = false; -pub const omit_stage2: bool = @ZIG_OMIT_STAGE2_BOOL@; diff --git a/src/link.zig b/src/link.zig index a0c0c5c369ee..a85969d61eb7 100644 --- a/src/link.zig +++ b/src/link.zig @@ -279,7 +279,7 @@ pub const File = struct { return &(try MachO.openPath(allocator, options)).base; } - const use_stage1 = build_options.is_stage1 and options.use_stage1; + const use_stage1 = build_options.have_stage1 and options.use_stage1; if (use_stage1 or options.emit == null) { return switch (options.target.ofmt) { .coff => &(try Coff.createEmpty(allocator, options)).base, @@ -817,7 +817,7 @@ pub const File = struct { // If there is no Zig code to compile, then we should skip flushing the output file // because it will not be part of the linker line anyway. const module_obj_path: ?[]const u8 = if (base.options.module) |module| blk: { - const use_stage1 = build_options.is_stage1 and base.options.use_stage1; + const use_stage1 = build_options.have_stage1 and base.options.use_stage1; if (use_stage1) { const obj_basename = try std.zig.binNameAlloc(arena, .{ .root_name = base.options.root_name, diff --git a/src/link/Coff.zig b/src/link/Coff.zig index 92e9dce3ad28..a01b9cf7c3ac 100644 --- a/src/link/Coff.zig +++ b/src/link/Coff.zig @@ -411,7 +411,7 @@ pub fn createEmpty(gpa: Allocator, options: link.Options) !*Coff { }; const use_llvm = build_options.have_llvm and options.use_llvm; - const use_stage1 = build_options.is_stage1 and options.use_stage1; + const use_stage1 = build_options.have_stage1 and options.use_stage1; if (use_llvm and !use_stage1) { self.llvm_object = try LlvmObject.create(gpa, options); } @@ -949,7 +949,7 @@ fn linkWithLLD(self: *Coff, comp: *Compilation, prog_node: *std.Progress.Node) ! // If there is no Zig code to compile, then we should skip flushing the output file because it // will not be part of the linker line anyway. const module_obj_path: ?[]const u8 = if (self.base.options.module) |module| blk: { - const use_stage1 = build_options.is_stage1 and self.base.options.use_stage1; + const use_stage1 = build_options.have_stage1 and self.base.options.use_stage1; if (use_stage1) { const obj_basename = try std.zig.binNameAlloc(arena, .{ .root_name = self.base.options.root_name, diff --git a/src/link/Elf.zig b/src/link/Elf.zig index 9902886baca5..ade4f62f91e2 100644 --- a/src/link/Elf.zig +++ b/src/link/Elf.zig @@ -328,7 +328,7 @@ pub fn createEmpty(gpa: Allocator, options: link.Options) !*Elf { .page_size = page_size, }; const use_llvm = build_options.have_llvm and options.use_llvm; - const use_stage1 = build_options.is_stage1 and options.use_stage1; + const use_stage1 = build_options.have_stage1 and options.use_stage1; if (use_llvm and !use_stage1) { self.llvm_object = try LlvmObject.create(gpa, options); } diff --git a/src/link/MachO.zig b/src/link/MachO.zig index 7a5084302363..020388cee871 100644 --- a/src/link/MachO.zig +++ b/src/link/MachO.zig @@ -272,7 +272,7 @@ pub const Export = struct { pub fn openPath(allocator: Allocator, options: link.Options) !*MachO { assert(options.target.ofmt == .macho); - const use_stage1 = build_options.is_stage1 and options.use_stage1; + const use_stage1 = build_options.have_stage1 and options.use_stage1; if (use_stage1 or options.emit == null) { return createEmpty(allocator, options); } @@ -363,7 +363,7 @@ pub fn createEmpty(gpa: Allocator, options: link.Options) !*MachO { const cpu_arch = options.target.cpu.arch; const page_size: u16 = if (cpu_arch == .aarch64) 0x4000 else 0x1000; const use_llvm = build_options.have_llvm and options.use_llvm; - const use_stage1 = build_options.is_stage1 and options.use_stage1; + const use_stage1 = build_options.have_stage1 and options.use_stage1; const self = try gpa.create(MachO); errdefer gpa.destroy(self); diff --git a/src/link/Wasm.zig b/src/link/Wasm.zig index 626786588873..626f17665215 100644 --- a/src/link/Wasm.zig +++ b/src/link/Wasm.zig @@ -356,7 +356,7 @@ pub fn createEmpty(gpa: Allocator, options: link.Options) !*Wasm { } const use_llvm = build_options.have_llvm and options.use_llvm; - const use_stage1 = build_options.is_stage1 and options.use_stage1; + const use_stage1 = build_options.have_stage1 and options.use_stage1; if (use_llvm and !use_stage1) { self.llvm_object = try LlvmObject.create(gpa, options); } @@ -2593,7 +2593,7 @@ fn linkWithLLD(self: *Wasm, comp: *Compilation, prog_node: *std.Progress.Node) ! // If there is no Zig code to compile, then we should skip flushing the output file because it // will not be part of the linker line anyway. const module_obj_path: ?[]const u8 = if (self.base.options.module) |mod| blk: { - const use_stage1 = build_options.is_stage1 and self.base.options.use_stage1; + const use_stage1 = build_options.have_stage1 and self.base.options.use_stage1; if (use_stage1) { const obj_basename = try std.zig.binNameAlloc(arena, .{ .root_name = self.base.options.root_name, @@ -2803,7 +2803,7 @@ fn linkWithLLD(self: *Wasm, comp: *Compilation, prog_node: *std.Progress.Node) ! if (self.base.options.module) |mod| { // when we use stage1, we use the exports that stage1 provided us. // For stage2, we can directly retrieve them from the module. - const use_stage1 = build_options.is_stage1 and self.base.options.use_stage1; + const use_stage1 = build_options.have_stage1 and self.base.options.use_stage1; if (use_stage1) { for (comp.export_symbol_names.items) |symbol_name| { try argv.append(try std.fmt.allocPrint(arena, "--export={s}", .{symbol_name})); diff --git a/src/main.zig b/src/main.zig index 91d171d23764..dd3a7e797bdb 100644 --- a/src/main.zig +++ b/src/main.zig @@ -2989,7 +2989,7 @@ fn buildOutputType( return std.io.getStdOut().writeAll(try comp.generateBuiltinZigSource(arena)); } if (arg_mode == .translate_c) { - const stage1_mode = use_stage1 orelse build_options.is_stage1; + const stage1_mode = use_stage1 orelse false; return cmdTranslateC(comp, arena, have_enable_cache, stage1_mode); } diff --git a/src/stage1.zig b/src/stage1.zig index 826fdd0f6bb9..e3f0daaa4450 100644 --- a/src/stage1.zig +++ b/src/stage1.zig @@ -18,7 +18,7 @@ const target_util = @import("target.zig"); comptime { assert(builtin.link_libc); - assert(build_options.is_stage1); + assert(build_options.have_stage1); assert(build_options.have_llvm); if (!builtin.is_test) { @export(main, .{ .name = "main" }); diff --git a/src/test.zig b/src/test.zig index a8567f24928b..babded13f9d7 100644 --- a/src/test.zig +++ b/src/test.zig @@ -25,7 +25,7 @@ const skip_stage1 = builtin.zig_backend != .stage1 or build_options.skip_stage1; const hr = "=" ** 80; test { - if (build_options.is_stage1) { + if (build_options.have_stage1) { @import("stage1.zig").os_init(); } diff --git a/test/cases/safety/empty slice with sentinel out of bounds.zig b/test/cases/safety/empty slice with sentinel out of bounds.zig index 835b084740c5..d989a33541c1 100644 --- a/test/cases/safety/empty slice with sentinel out of bounds.zig +++ b/test/cases/safety/empty slice with sentinel out of bounds.zig @@ -2,7 +2,7 @@ const std = @import("std"); pub fn panic(message: []const u8, stack_trace: ?*std.builtin.StackTrace) noreturn { _ = stack_trace; - if (std.mem.eql(u8, message, "attempt to index out of bound: index 1, len 0")) { + if (std.mem.eql(u8, message, "index out of bounds: index 1, len 0")) { std.process.exit(0); } std.process.exit(1); diff --git a/test/cases/safety/out of bounds slice access.zig b/test/cases/safety/out of bounds slice access.zig index a30532aee78c..ddd9e74cf285 100644 --- a/test/cases/safety/out of bounds slice access.zig +++ b/test/cases/safety/out of bounds slice access.zig @@ -2,20 +2,20 @@ const std = @import("std"); pub fn panic(message: []const u8, stack_trace: ?*std.builtin.StackTrace) noreturn { _ = stack_trace; - if (std.mem.eql(u8, message, "attempt to index out of bound: index 4, len 4")) { + if (std.mem.eql(u8, message, "index out of bounds: index 4, len 4")) { std.process.exit(0); } std.process.exit(1); } pub fn main() !void { - const a = [_]i32{1, 2, 3, 4}; + const a = [_]i32{ 1, 2, 3, 4 }; baz(bar(&a)); return error.TestFailed; } fn bar(a: []const i32) i32 { return a[4]; } -fn baz(_: i32) void { } +fn baz(_: i32) void {} // run // backend=llvm // target=native diff --git a/test/cases/safety/slice with sentinel out of bounds - runtime len.zig b/test/cases/safety/slice with sentinel out of bounds - runtime len.zig index fa2e127107c8..524c69d7b7dd 100644 --- a/test/cases/safety/slice with sentinel out of bounds - runtime len.zig +++ b/test/cases/safety/slice with sentinel out of bounds - runtime len.zig @@ -2,7 +2,7 @@ const std = @import("std"); pub fn panic(message: []const u8, stack_trace: ?*std.builtin.StackTrace) noreturn { _ = stack_trace; - if (std.mem.eql(u8, message, "attempt to index out of bound: index 5, len 4")) { + if (std.mem.eql(u8, message, "index out of bounds: index 5, len 4")) { std.process.exit(0); } std.process.exit(1); diff --git a/test/cases/safety/slice with sentinel out of bounds.zig b/test/cases/safety/slice with sentinel out of bounds.zig index 3cc8bfb355a4..636235a5b34c 100644 --- a/test/cases/safety/slice with sentinel out of bounds.zig +++ b/test/cases/safety/slice with sentinel out of bounds.zig @@ -2,7 +2,7 @@ const std = @import("std"); pub fn panic(message: []const u8, stack_trace: ?*std.builtin.StackTrace) noreturn { _ = stack_trace; - if (std.mem.eql(u8, message, "attempt to index out of bound: index 5, len 4")) { + if (std.mem.eql(u8, message, "index out of bounds: index 5, len 4")) { std.process.exit(0); } std.process.exit(1); diff --git a/test/tests.zig b/test/tests.zig index 0bca9a9d1603..a622521182ed 100644 --- a/test/tests.zig +++ b/test/tests.zig @@ -605,7 +605,6 @@ pub fn addPkgTests( skip_libc: bool, skip_stage1: bool, skip_stage2: bool, - is_stage1: bool, ) *build.Step { const step = b.step(b.fmt("test-{s}", .{name}), desc); @@ -634,7 +633,7 @@ pub fn addPkgTests( if (test_target.backend) |backend| switch (backend) { .stage1 => if (skip_stage1) continue, else => if (skip_stage2) continue, - } else if (is_stage1 and skip_stage1) continue; + } else if (skip_stage2) continue; const want_this_mode = for (modes) |m| { if (m == test_target.mode) break true; @@ -924,7 +923,7 @@ pub const StackTracesContext = struct { pos = marks[i] + delim.len; } // locate source basename - pos = mem.lastIndexOfScalar(u8, line[0..marks[0]], fs.path.sep) orelse { + pos = mem.lastIndexOfAny(u8, line[0..marks[0]], "\\/") orelse { // unexpected pattern: emit raw line and cont try buf.appendSlice(line); try buf.appendSlice("\n"); @@ -936,9 +935,9 @@ pub const StackTracesContext = struct { try buf.appendSlice(line[pos + 1 .. marks[2] + delims[2].len]); try buf.appendSlice(" [address]"); if (self.mode == .Debug) { - if (mem.lastIndexOfScalar(u8, line[marks[4]..marks[5]], '.')) |idot| { - // On certain platforms (windows) or possibly depending on how we choose to link main - // the object file extension may be present so we simply strip any extension. + // On certain platforms (windows) or possibly depending on how we choose to link main + // the object file extension may be present so we simply strip any extension. + if (mem.indexOfScalar(u8, line[marks[4]..marks[5]], '.')) |idot| { try buf.appendSlice(line[marks[3] .. marks[4] + idot]); try buf.appendSlice(line[marks[5]..]); } else {