aboutsummaryrefslogtreecommitdiff
path: root/src/main.cpp
diff options
context:
space:
mode:
authorgingerBill <gingerBill@users.noreply.github.com>2024-07-19 11:43:56 +0100
committerGitHub <noreply@github.com>2024-07-19 11:43:56 +0100
commit62f455f47bfa0cfc90047a389ae5959312057a7d (patch)
treeb7c619d8e31ee0f325f16e23a4494325b5975790 /src/main.cpp
parent37b026cb9bdd29aa657a54b76d2595bef40ff8c8 (diff)
parent163287d9ce6b998d65ea2112144bbc9288fa9efa (diff)
Merge branch 'master' into syscall-fix
Diffstat (limited to 'src/main.cpp')
-rw-r--r--src/main.cpp688
1 files changed, 638 insertions, 50 deletions
diff --git a/src/main.cpp b/src/main.cpp
index ee7de7f81..00734c050 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -61,6 +61,7 @@ gb_global Timings global_timings = {0};
#include "llvm-c/Types.h"
#else
#include <llvm-c/Types.h>
+#include <signal.h>
#endif
#include "parser.hpp"
@@ -70,6 +71,8 @@ gb_global Timings global_timings = {0};
#include "checker.cpp"
#include "docs.cpp"
+#include "cached.cpp"
+
#include "linker.cpp"
#if defined(GB_SYSTEM_WINDOWS) && defined(ODIN_TILDE_BACKEND)
@@ -86,27 +89,46 @@ gb_global Timings global_timings = {0};
#if defined(GB_SYSTEM_OSX)
#include <llvm/Config/llvm-config.h>
- #if LLVM_VERSION_MAJOR < 11
- #error LLVM Version 11+ is required => "brew install llvm@11"
- #endif
- #if (LLVM_VERSION_MAJOR > 14 && LLVM_VERSION_MAJOR < 17) || LLVM_VERSION_MAJOR > 17
- #error LLVM Version 11..=14 or =17 is required => "brew install llvm@14"
+ #if LLVM_VERSION_MAJOR < 11 || (LLVM_VERSION_MAJOR > 14 && LLVM_VERSION_MAJOR < 17) || LLVM_VERSION_MAJOR > 18
+ #error LLVM Version 11..=14 or =18 is required => "brew install llvm@14"
#endif
#endif
#include "bug_report.cpp"
// NOTE(bill): 'name' is used in debugging and profiling modes
-gb_internal i32 system_exec_command_line_app(char const *name, char const *fmt, ...) {
+gb_internal i32 system_exec_command_line_app_internal(bool exit_on_err, char const *name, char const *fmt, va_list va) {
isize const cmd_cap = 64<<20; // 64 MiB should be more than enough
char *cmd_line = gb_alloc_array(gb_heap_allocator(), char, cmd_cap);
isize cmd_len = 0;
- va_list va;
i32 exit_code = 0;
- va_start(va, fmt);
cmd_len = gb_snprintf_va(cmd_line, cmd_cap-1, fmt, va);
- va_end(va);
+
+ if (build_context.print_linker_flags) {
+ // NOTE(bill): remove the first argument (the executable) from the executable list
+ // and then print it for the "linker flags"
+ while (*cmd_line && gb_char_is_space(*cmd_line)) {
+ cmd_line++;
+ }
+ if (*cmd_line == '\"') for (cmd_line++; *cmd_line; cmd_line++) {
+ if (*cmd_line == '\\') {
+ cmd_line++;
+ if (*cmd_line == '\"') {
+ cmd_line++;
+ }
+ } else if (*cmd_line == '\"') {
+ cmd_line++;
+ break;
+ }
+ }
+ while (*cmd_line && gb_char_is_space(*cmd_line)) {
+ cmd_line++;
+ }
+
+ fprintf(stdout, "%s\n", cmd_line);
+ return exit_code;
+ }
#if defined(GB_SYSTEM_WINDOWS)
STARTUPINFOW start_info = {gb_size_of(STARTUPINFOW)};
@@ -146,18 +168,68 @@ gb_internal i32 system_exec_command_line_app(char const *name, char const *fmt,
gb_printf_err("%s\n\n", cmd_line);
}
exit_code = system(cmd_line);
+ if (exit_on_err && WIFSIGNALED(exit_code)) {
+ raise(WTERMSIG(exit_code));
+ }
if (WIFEXITED(exit_code)) {
exit_code = WEXITSTATUS(exit_code);
}
#endif
- if (exit_code) {
+ if (exit_on_err && exit_code) {
exit(exit_code);
}
return exit_code;
}
+gb_internal i32 system_exec_command_line_app(char const *name, char const *fmt, ...) {
+ va_list va;
+ va_start(va, fmt);
+ i32 exit_code = system_exec_command_line_app_internal(/* exit_on_err= */ false, name, fmt, va);
+ va_end(va);
+ return exit_code;
+}
+
+gb_internal void system_must_exec_command_line_app(char const *name, char const *fmt, ...) {
+ va_list va;
+ va_start(va, fmt);
+ system_exec_command_line_app_internal(/* exit_on_err= */ true, name, fmt, va);
+ va_end(va);
+}
+
+#if defined(GB_SYSTEM_WINDOWS)
+#define popen _popen
+#define pclose _pclose
+#endif
+
+gb_internal bool system_exec_command_line_app_output(char const *command, gbString *output) {
+ GB_ASSERT(output);
+
+ u8 buffer[256];
+ FILE *stream;
+ stream = popen(command, "r");
+ if (!stream) {
+ return false;
+ }
+ defer (pclose(stream));
+
+ while (!feof(stream)) {
+ size_t n = fread(buffer, 1, 255, stream);
+ *output = gb_string_append_length(*output, buffer, n);
+
+ if (ferror(stream)) {
+ return false;
+ }
+ }
+
+ if (build_context.show_system_calls) {
+ gb_printf_err("[SYSTEM CALL OUTPUT] %s -> %s\n", command, *output);
+ }
+
+ return true;
+}
+
gb_internal Array<String> setup_args(int argc, char const **argv) {
gbAllocator a = heap_allocator();
@@ -237,6 +309,8 @@ enum BuildFlagKind {
BuildFlag_ShowMoreTimings,
BuildFlag_ExportTimings,
BuildFlag_ExportTimingsFile,
+ BuildFlag_ExportDependencies,
+ BuildFlag_ExportDependenciesFile,
BuildFlag_ShowSystemCalls,
BuildFlag_ThreadCount,
BuildFlag_KeepTempFiles,
@@ -257,6 +331,9 @@ enum BuildFlagKind {
BuildFlag_NoThreadedChecker,
BuildFlag_ShowDebugMessages,
+ BuildFlag_ShowDefineables,
+ BuildFlag_ExportDefineables,
+
BuildFlag_Vet,
BuildFlag_VetShadowing,
BuildFlag_VetUnused,
@@ -266,20 +343,22 @@ enum BuildFlagKind {
BuildFlag_VetUsingParam,
BuildFlag_VetStyle,
BuildFlag_VetSemicolon,
+ BuildFlag_VetCast,
+ BuildFlag_VetTabs,
+ BuildFlag_CustomAttribute,
BuildFlag_IgnoreUnknownAttributes,
BuildFlag_ExtraLinkerFlags,
BuildFlag_ExtraAssemblerFlags,
BuildFlag_Microarch,
BuildFlag_TargetFeatures,
+ BuildFlag_StrictTargetFeatures,
BuildFlag_MinimumOSVersion,
BuildFlag_NoThreadLocal,
BuildFlag_RelocMode,
BuildFlag_DisableRedZone,
- BuildFlag_TestName,
-
BuildFlag_DisallowDo,
BuildFlag_DefaultToNilAllocator,
BuildFlag_DefaultToPanicAllocator,
@@ -307,9 +386,15 @@ enum BuildFlagKind {
BuildFlag_MinLinkLibs,
+ BuildFlag_PrintLinkerFlags,
+
// internal use only
BuildFlag_InternalIgnoreLazy,
BuildFlag_InternalIgnoreLLVMBuild,
+ BuildFlag_InternalIgnorePanic,
+ BuildFlag_InternalModulePerFile,
+ BuildFlag_InternalCached,
+ BuildFlag_InternalNoInline,
BuildFlag_Tilde,
@@ -322,7 +407,6 @@ enum BuildFlagKind {
BuildFlag_Subsystem,
#endif
-
BuildFlag_COUNT,
};
@@ -429,6 +513,8 @@ gb_internal bool parse_build_flags(Array<String> args) {
add_flag(&build_flags, BuildFlag_ShowMoreTimings, str_lit("show-more-timings"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_ExportTimings, str_lit("export-timings"), BuildFlagParam_String, Command__does_check);
add_flag(&build_flags, BuildFlag_ExportTimingsFile, str_lit("export-timings-file"), BuildFlagParam_String, Command__does_check);
+ add_flag(&build_flags, BuildFlag_ExportDependencies, str_lit("export-dependencies"), BuildFlagParam_String, Command__does_build);
+ add_flag(&build_flags, BuildFlag_ExportDependenciesFile, str_lit("export-dependencies-file"), BuildFlagParam_String, Command__does_build);
add_flag(&build_flags, BuildFlag_ShowUnused, str_lit("show-unused"), BuildFlagParam_None, Command_check);
add_flag(&build_flags, BuildFlag_ShowUnusedWithLocation, str_lit("show-unused-with-location"), BuildFlagParam_None, Command_check);
add_flag(&build_flags, BuildFlag_ShowSystemCalls, str_lit("show-system-calls"), BuildFlagParam_None, Command_all);
@@ -452,6 +538,9 @@ gb_internal bool parse_build_flags(Array<String> args) {
add_flag(&build_flags, BuildFlag_NoThreadedChecker, str_lit("no-threaded-checker"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_ShowDebugMessages, str_lit("show-debug-messages"), BuildFlagParam_None, Command_all);
+ add_flag(&build_flags, BuildFlag_ShowDefineables, str_lit("show-defineables"), BuildFlagParam_None, Command__does_check);
+ add_flag(&build_flags, BuildFlag_ExportDefineables, str_lit("export-defineables"), BuildFlagParam_String, Command__does_check);
+
add_flag(&build_flags, BuildFlag_Vet, str_lit("vet"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_VetUnused, str_lit("vet-unused"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_VetUnusedVariables, str_lit("vet-unused-variables"), BuildFlagParam_None, Command__does_check);
@@ -461,19 +550,21 @@ gb_internal bool parse_build_flags(Array<String> args) {
add_flag(&build_flags, BuildFlag_VetUsingParam, str_lit("vet-using-param"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_VetStyle, str_lit("vet-style"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_VetSemicolon, str_lit("vet-semicolon"), BuildFlagParam_None, Command__does_check);
+ add_flag(&build_flags, BuildFlag_VetCast, str_lit("vet-cast"), BuildFlagParam_None, Command__does_check);
+ add_flag(&build_flags, BuildFlag_VetTabs, str_lit("vet-tabs"), BuildFlagParam_None, Command__does_check);
+ add_flag(&build_flags, BuildFlag_CustomAttribute, str_lit("custom-attribute"), BuildFlagParam_String, Command__does_check, true);
add_flag(&build_flags, BuildFlag_IgnoreUnknownAttributes, str_lit("ignore-unknown-attributes"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_ExtraLinkerFlags, str_lit("extra-linker-flags"), BuildFlagParam_String, Command__does_build);
add_flag(&build_flags, BuildFlag_ExtraAssemblerFlags, str_lit("extra-assembler-flags"), BuildFlagParam_String, Command__does_build);
add_flag(&build_flags, BuildFlag_Microarch, str_lit("microarch"), BuildFlagParam_String, Command__does_build);
add_flag(&build_flags, BuildFlag_TargetFeatures, str_lit("target-features"), BuildFlagParam_String, Command__does_build);
+ add_flag(&build_flags, BuildFlag_StrictTargetFeatures, str_lit("strict-target-features"), BuildFlagParam_None, Command__does_build);
add_flag(&build_flags, BuildFlag_MinimumOSVersion, str_lit("minimum-os-version"), BuildFlagParam_String, Command__does_build);
add_flag(&build_flags, BuildFlag_RelocMode, str_lit("reloc-mode"), BuildFlagParam_String, Command__does_build);
add_flag(&build_flags, BuildFlag_DisableRedZone, str_lit("disable-red-zone"), BuildFlagParam_None, Command__does_build);
- add_flag(&build_flags, BuildFlag_TestName, str_lit("test-name"), BuildFlagParam_String, Command_test);
-
add_flag(&build_flags, BuildFlag_DisallowDo, str_lit("disallow-do"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_DefaultToNilAllocator, str_lit("default-to-nil-allocator"), BuildFlagParam_None, Command__does_check);
add_flag(&build_flags, BuildFlag_DefaultToPanicAllocator, str_lit("default-to-panic-allocator"),BuildFlagParam_None, Command__does_check);
@@ -501,8 +592,14 @@ gb_internal bool parse_build_flags(Array<String> args) {
add_flag(&build_flags, BuildFlag_MinLinkLibs, str_lit("min-link-libs"), BuildFlagParam_None, Command__does_build);
+ add_flag(&build_flags, BuildFlag_PrintLinkerFlags, str_lit("print-linker-flags"), BuildFlagParam_None, Command_build);
+
add_flag(&build_flags, BuildFlag_InternalIgnoreLazy, str_lit("internal-ignore-lazy"), BuildFlagParam_None, Command_all);
add_flag(&build_flags, BuildFlag_InternalIgnoreLLVMBuild, str_lit("internal-ignore-llvm-build"),BuildFlagParam_None, Command_all);
+ add_flag(&build_flags, BuildFlag_InternalIgnorePanic, str_lit("internal-ignore-panic"), BuildFlagParam_None, Command_all);
+ add_flag(&build_flags, BuildFlag_InternalModulePerFile, str_lit("internal-module-per-file"), BuildFlagParam_None, Command_all);
+ add_flag(&build_flags, BuildFlag_InternalCached, str_lit("internal-cached"), BuildFlagParam_None, Command_all);
+ add_flag(&build_flags, BuildFlag_InternalNoInline, str_lit("internal-no-inline"), BuildFlagParam_None, Command_all);
#if ALLOW_TILDE
add_flag(&build_flags, BuildFlag_Tilde, str_lit("tilde"), BuildFlagParam_None, Command__does_build);
@@ -510,6 +607,7 @@ gb_internal bool parse_build_flags(Array<String> args) {
add_flag(&build_flags, BuildFlag_Sanitize, str_lit("sanitize"), BuildFlagParam_String, Command__does_build, true);
+
#if defined(GB_SYSTEM_WINDOWS)
add_flag(&build_flags, BuildFlag_IgnoreVsSearch, str_lit("ignore-vs-search"), BuildFlagParam_None, Command__does_build);
add_flag(&build_flags, BuildFlag_ResourceFile, str_lit("resource"), BuildFlagParam_String, Command__does_build);
@@ -754,6 +852,53 @@ gb_internal bool parse_build_flags(Array<String> args) {
break;
}
+ case BuildFlag_ExportDependencies: {
+ GB_ASSERT(value.kind == ExactValue_String);
+
+ if (value.value_string == "make") {
+ build_context.export_dependencies_format = DependenciesExportMake;
+ } else if (value.value_string == "json") {
+ build_context.export_dependencies_format = DependenciesExportJson;
+ } else {
+ gb_printf_err("Invalid export format for -export-dependencies:<string>, got %.*s\n", LIT(value.value_string));
+ gb_printf_err("Valid export formats:\n");
+ gb_printf_err("\tmake\n");
+ gb_printf_err("\tjson\n");
+ bad_flags = true;
+ }
+ break;
+ }
+ case BuildFlag_ExportDependenciesFile: {
+ GB_ASSERT(value.kind == ExactValue_String);
+
+ String export_path = string_trim_whitespace(value.value_string);
+ if (is_build_flag_path_valid(export_path)) {
+ build_context.export_dependencies_file = path_to_full_path(heap_allocator(), export_path);
+ } else {
+ gb_printf_err("Invalid -export-dependencies path, got %.*s\n", LIT(export_path));
+ bad_flags = true;
+ }
+
+ break;
+ }
+ case BuildFlag_ShowDefineables: {
+ GB_ASSERT(value.kind == ExactValue_Invalid);
+ build_context.show_defineables = true;
+ break;
+ }
+ case BuildFlag_ExportDefineables: {
+ GB_ASSERT(value.kind == ExactValue_String);
+
+ String export_path = string_trim_whitespace(value.value_string);
+ if (is_build_flag_path_valid(export_path)) {
+ build_context.export_defineables_file = path_to_full_path(heap_allocator(), export_path);
+ } else {
+ gb_printf_err("Invalid -export-defineables path, got %.*s\n", LIT(export_path));
+ bad_flags = true;
+ }
+
+ break;
+ }
case BuildFlag_ShowSystemCalls: {
GB_ASSERT(value.kind == ExactValue_Invalid);
build_context.show_system_calls = true;
@@ -990,20 +1135,27 @@ gb_internal bool parse_build_flags(Array<String> args) {
build_context.build_mode = BuildMode_DynamicLibrary;
} else if (str == "obj" || str == "object") {
build_context.build_mode = BuildMode_Object;
+ } else if (str == "static" || str == "lib") {
+ build_context.build_mode = BuildMode_StaticLibrary;
} else if (str == "exe") {
build_context.build_mode = BuildMode_Executable;
} else if (str == "asm" || str == "assembly" || str == "assembler") {
build_context.build_mode = BuildMode_Assembly;
} else if (str == "llvm" || str == "llvm-ir") {
build_context.build_mode = BuildMode_LLVM_IR;
+ } else if (str == "test") {
+ build_context.build_mode = BuildMode_Executable;
+ build_context.command_kind = Command_test;
} else {
gb_printf_err("Unknown build mode '%.*s'\n", LIT(str));
gb_printf_err("Valid build modes:\n");
gb_printf_err("\tdll, shared, dynamic\n");
+ gb_printf_err("\tlib, static\n");
gb_printf_err("\tobj, object\n");
gb_printf_err("\texe\n");
gb_printf_err("\tasm, assembly, assembler\n");
gb_printf_err("\tllvm, llvm-ir\n");
+ gb_printf_err("\ttest\n");
bad_flags = true;
break;
}
@@ -1059,6 +1211,31 @@ gb_internal bool parse_build_flags(Array<String> args) {
case BuildFlag_VetUsingParam: build_context.vet_flags |= VetFlag_UsingParam; break;
case BuildFlag_VetStyle: build_context.vet_flags |= VetFlag_Style; break;
case BuildFlag_VetSemicolon: build_context.vet_flags |= VetFlag_Semicolon; break;
+ case BuildFlag_VetCast: build_context.vet_flags |= VetFlag_Cast; break;
+ case BuildFlag_VetTabs: build_context.vet_flags |= VetFlag_Tabs; break;
+
+ case BuildFlag_CustomAttribute:
+ {
+ GB_ASSERT(value.kind == ExactValue_String);
+ String val = value.value_string;
+ String_Iterator it = {val, 0};
+ for (;;) {
+ String attr = string_split_iterator(&it, ',');
+ if (attr.len == 0) {
+ break;
+ }
+
+ attr = string_trim_whitespace(attr);
+ if (!string_is_valid_identifier(attr)) {
+ gb_printf_err("-custom-attribute '%.*s' must be a valid identifier\n", LIT(attr));
+ bad_flags = true;
+ continue;
+ }
+
+ string_set_add(&build_context.custom_attributes, attr);
+ }
+ }
+ break;
case BuildFlag_IgnoreUnknownAttributes:
build_context.ignore_unknown_attributes = true;
@@ -1083,6 +1260,9 @@ gb_internal bool parse_build_flags(Array<String> args) {
string_to_lower(&build_context.target_features_string);
break;
}
+ case BuildFlag_StrictTargetFeatures:
+ build_context.strict_target_features = true;
+ break;
case BuildFlag_MinimumOSVersion: {
GB_ASSERT(value.kind == ExactValue_String);
build_context.minimum_os_version_string = value.value_string;
@@ -1114,21 +1294,6 @@ gb_internal bool parse_build_flags(Array<String> args) {
case BuildFlag_DisableRedZone:
build_context.disable_red_zone = true;
break;
- case BuildFlag_TestName: {
- GB_ASSERT(value.kind == ExactValue_String);
- {
- String name = value.value_string;
- if (!string_is_valid_identifier(name)) {
- gb_printf_err("Test name '%.*s' must be a valid identifier\n", LIT(name));
- bad_flags = true;
- break;
- }
- string_set_add(&build_context.test_names, name);
-
- // NOTE(bill): Allow for multiple -test-name
- continue;
- }
- }
case BuildFlag_DisallowDo:
build_context.disallow_do = true;
break;
@@ -1239,12 +1404,31 @@ gb_internal bool parse_build_flags(Array<String> args) {
build_context.min_link_libs = true;
break;
+ case BuildFlag_PrintLinkerFlags:
+ build_context.print_linker_flags = true;
+ break;
+
case BuildFlag_InternalIgnoreLazy:
build_context.ignore_lazy = true;
break;
case BuildFlag_InternalIgnoreLLVMBuild:
build_context.ignore_llvm_build = true;
break;
+ case BuildFlag_InternalIgnorePanic:
+ build_context.ignore_panic = true;
+ break;
+ case BuildFlag_InternalModulePerFile:
+ build_context.module_per_file = true;
+ build_context.use_separate_modules = true;
+ break;
+ case BuildFlag_InternalCached:
+ build_context.cached = true;
+ build_context.use_separate_modules = true;
+ break;
+ case BuildFlag_InternalNoInline:
+ build_context.internal_no_inline = true;
+ break;
+
case BuildFlag_Tilde:
build_context.tilde_backend = true;
break;
@@ -1264,6 +1448,7 @@ gb_internal bool parse_build_flags(Array<String> args) {
}
break;
+
#if defined(GB_SYSTEM_WINDOWS)
case BuildFlag_IgnoreVsSearch: {
GB_ASSERT(value.kind == ExactValue_Invalid);
@@ -1275,8 +1460,9 @@ gb_internal bool parse_build_flags(Array<String> args) {
String path = value.value_string;
path = string_trim_whitespace(path);
if (is_build_flag_path_valid(path)) {
- if(!string_ends_with(path, str_lit(".rc"))) {
- gb_printf_err("Invalid -resource path %.*s, missing .rc\n", LIT(path));
+ bool is_resource = string_ends_with(path, str_lit(".rc")) || string_ends_with(path, str_lit(".res"));
+ if(!is_resource) {
+ gb_printf_err("Invalid -resource path %.*s, missing .rc or .res file\n", LIT(path));
bad_flags = true;
break;
} else if (!gb_file_exists((const char *)path.text)) {
@@ -1404,6 +1590,16 @@ gb_internal bool parse_build_flags(Array<String> args) {
gb_printf_err("`-export-timings:<format>` requires `-show-timings` or `-show-more-timings` to be present\n");
bad_flags = true;
}
+
+
+ if (build_context.export_dependencies_format != DependenciesExportUnspecified && build_context.print_linker_flags) {
+ gb_printf_err("-export-dependencies cannot be used with -print-linker-flags\n");
+ bad_flags = true;
+ } else if (build_context.show_timings && build_context.print_linker_flags) {
+ gb_printf_err("-show-timings/-show-more-timings cannot be used with -print-linker-flags\n");
+ bad_flags = true;
+ }
+
return !bad_flags;
}
@@ -1502,6 +1698,115 @@ gb_internal void timings_export_all(Timings *t, Checker *c, bool timings_are_fin
gb_printf("Done.\n");
}
+gb_internal void check_defines(BuildContext *bc, Checker *c) {
+ for (auto const &entry : bc->defined_values) {
+ String name = make_string_c(entry.key);
+ ExactValue value = entry.value;
+ GB_ASSERT(value.kind != ExactValue_Invalid);
+
+ bool found = false;
+ for_array(i, c->info.defineables) {
+ Defineable *def = &c->info.defineables[i];
+ if (def->name == name) {
+ found = true;
+ break;
+ }
+ }
+
+ if (!found) {
+ ERROR_BLOCK();
+ warning(nullptr, "given -define:%.*s is unused in the project", LIT(name));
+ error_line("\tSuggestion: use the -show-defineables flag for an overview of the possible defines\n");
+ }
+ }
+}
+
+gb_internal void temp_alloc_defineable_strings(Checker *c) {
+ for_array(i, c->info.defineables) {
+ Defineable *def = &c->info.defineables[i];
+ def->default_value_str = make_string_c(write_exact_value_to_string(gb_string_make(temporary_allocator(), ""), def->default_value));
+ def->pos_str = make_string_c(token_pos_to_string(def->pos));
+ }
+}
+
+gb_internal GB_COMPARE_PROC(defineables_cmp) {
+ Defineable *x = (Defineable *)a;
+ Defineable *y = (Defineable *)b;
+
+ int cmp = 0;
+
+ String x_file = get_file_path_string(x->pos.file_id);
+ String y_file = get_file_path_string(y->pos.file_id);
+ cmp = string_compare(x_file, y_file);
+ if (cmp) {
+ return cmp;
+ }
+
+ return i32_cmp(x->pos.offset, y->pos.offset);
+}
+
+gb_internal void sort_defineables(Checker *c) {
+ gb_sort_array(c->info.defineables.data, c->info.defineables.count, defineables_cmp);
+}
+
+gb_internal void export_defineables(Checker *c, String path) {
+ gbFile f = {};
+ gbFileError err = gb_file_open_mode(&f, gbFileMode_Write, (char *)path.text);
+ if (err != gbFileError_None) {
+ gb_printf_err("Failed to export defineables to: %.*s\n", LIT(path));
+ gb_exit(1);
+ return;
+ } else {
+ gb_printf("Exporting defineables to '%.*s'...\n", LIT(path));
+ }
+ defer (gb_file_close(&f));
+
+ gbString docs = gb_string_make(heap_allocator(), "");
+ defer (gb_string_free(docs));
+
+ gb_fprintf(&f, "Defineable,Default Value,Docs,Location\n");
+ for_array(i, c->info.defineables) {
+ Defineable *def = &c->info.defineables[i];
+
+ gb_string_clear(docs);
+ if (def->docs) {
+ docs = gb_string_appendc(docs, "\"");
+ for (Token const &token : def->docs->list) {
+ for (isize i = 0; i < token.string.len; i++) {
+ u8 c = token.string.text[i];
+ if (c == '"') {
+ docs = gb_string_appendc(docs, "\"\"");
+ } else {
+ docs = gb_string_append_length(docs, &c, 1);
+ }
+ }
+ }
+ docs = gb_string_appendc(docs, "\"");
+ }
+
+ gb_fprintf(&f,"%.*s,%.*s,%s,%.*s\n", LIT(def->name), LIT(def->default_value_str), docs, LIT(def->pos_str));
+ }
+}
+
+gb_internal void show_defineables(Checker *c) {
+ for_array(i, c->info.defineables) {
+ Defineable *def = &c->info.defineables[i];
+ if (has_ansi_terminal_colours()) {
+ gb_printf("\x1b[0;90m");
+ }
+ printf("%.*s\n", LIT(def->pos_str));
+ if (def->docs) {
+ for (Token const &token : def->docs->list) {
+ gb_printf("%.*s\n", LIT(token.string));
+ }
+ }
+ if (has_ansi_terminal_colours()) {
+ gb_printf("\x1b[0m");
+ }
+ gb_printf("%.*s :: %.*s\n\n", LIT(def->name), LIT(def->default_value_str));
+ }
+}
+
gb_internal void show_timings(Checker *c, Timings *t) {
Parser *p = c->parser;
isize lines = p->total_line_count;
@@ -1630,11 +1935,123 @@ gb_internal void show_timings(Checker *c, Timings *t) {
}
}
+gb_internal GB_COMPARE_PROC(file_path_cmp) {
+ AstFile *x = *(AstFile **)a;
+ AstFile *y = *(AstFile **)b;
+ return string_compare(x->fullpath, y->fullpath);
+}
+
+gb_internal void export_dependencies(Checker *c) {
+ GB_ASSERT(build_context.export_dependencies_format != DependenciesExportUnspecified);
+
+ if (build_context.export_dependencies_file.len <= 0) {
+ gb_printf_err("No dependency file specified with `-export-dependencies-file`\n");
+ exit_with_errors();
+ return;
+ }
+
+ Parser *p = c->parser;
+
+ gbFile f = {};
+ char * fileName = (char *)build_context.export_dependencies_file.text;
+ gbFileError err = gb_file_open_mode(&f, gbFileMode_Write, fileName);
+ if (err != gbFileError_None) {
+ gb_printf_err("Failed to export dependencies to: %s\n", fileName);
+ exit_with_errors();
+ return;
+ }
+ defer (gb_file_close(&f));
+
+
+ auto files = array_make<AstFile *>(heap_allocator());
+ for (AstPackage *pkg : p->packages) {
+ for (AstFile *f : pkg->files) {
+ array_add(&files, f);
+ }
+ }
+ array_sort(files, file_path_cmp);
+
+
+ auto load_files = array_make<LoadFileCache *>(heap_allocator());
+ for (auto const &entry : c->info.load_file_cache) {
+ auto *cache = entry.value;
+ if (!cache || !cache->exists) {
+ continue;
+ }
+ array_add(&load_files, cache);
+ }
+ array_sort(files, file_cache_sort_cmp);
+
+ if (build_context.export_dependencies_format == DependenciesExportMake) {
+ String exe_name = path_to_string(heap_allocator(), build_context.build_paths[BuildPath_Output]);
+ defer (gb_free(heap_allocator(), exe_name.text));
+
+ gb_fprintf(&f, "%.*s:", LIT(exe_name));
+
+ isize current_line_length = exe_name.len + 1;
+
+ for_array(i, files) {
+ AstFile *file = files[i];
+ /* Arbitrary line break value. Maybe make this better? */
+ if (current_line_length >= 80-2) {
+ gb_file_write(&f, " \\\n ", 4);
+ current_line_length = 1;
+ }
+
+ gb_file_write(&f, " ", 1);
+ current_line_length++;
+
+ for (isize k = 0; k < file->fullpath.len; k++) {
+ char part = file->fullpath.text[k];
+ if (part == ' ') {
+ gb_file_write(&f, "\\", 1);
+ current_line_length++;
+ }
+ gb_file_write(&f, &part, 1);
+ current_line_length++;
+ }
+ }
+
+ gb_fprintf(&f, "\n");
+ } else if (build_context.export_dependencies_format == DependenciesExportJson) {
+ gb_fprintf(&f, "{\n");
+
+ gb_fprintf(&f, "\t\"source_files\": [\n");
+
+ for_array(i, files) {
+ AstFile *file = files[i];
+ gb_fprintf(&f, "\t\t\"%.*s\"", LIT(file->fullpath));
+ if (i+1 == files.count) {
+ gb_fprintf(&f, ",");
+ }
+ gb_fprintf(&f, "\n");
+ }
+
+ gb_fprintf(&f, "\t],\n");
+
+ gb_fprintf(&f, "\t\"load_files\": [\n");
+
+ for_array(i, load_files) {
+ LoadFileCache *cache = load_files[i];
+ gb_fprintf(&f, "\t\t\"%.*s\"", LIT(cache->path));
+ if (i+1 == load_files.count) {
+ gb_fprintf(&f, ",");
+ }
+ gb_fprintf(&f, "\n");
+ }
+
+ gb_fprintf(&f, "\t]\n");
+
+ gb_fprintf(&f, "}\n");
+ }
+}
+
gb_internal void remove_temp_files(lbGenerator *gen) {
if (build_context.keep_temp_files) return;
switch (build_context.build_mode) {
case BuildMode_Executable:
+ case BuildMode_StaticLibrary:
case BuildMode_DynamicLibrary:
break;
@@ -1653,6 +2070,7 @@ gb_internal void remove_temp_files(lbGenerator *gen) {
if (!build_context.keep_object_files) {
switch (build_context.build_mode) {
case BuildMode_Executable:
+ case BuildMode_StaticLibrary:
case BuildMode_DynamicLibrary:
for (String const &path : gen->output_object_paths) {
gb_file_remove(cast(char const *)path.text);
@@ -1754,7 +2172,7 @@ gb_internal void print_show_help(String const arg0, String const &command) {
if (LB_USE_NEW_PASS_SYSTEM) {
print_usage_line(3, "-o:aggressive");
}
- print_usage_line(2, "The default is -o:minimal.");
+ print_usage_line(2, "The default is -o:none.");
print_usage_line(0, "");
}
@@ -1783,6 +2201,18 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(2, "Example: -export-timings-file:timings.json");
print_usage_line(0, "");
+ print_usage_line(1, "-export-dependencies:<format>");
+ print_usage_line(2, "Exports dependencies to one of a few formats. Requires `-export-dependencies-file`.");
+ print_usage_line(2, "Available options:");
+ print_usage_line(3, "-export-dependencies:make Exports in Makefile format");
+ print_usage_line(3, "-export-dependencies:json Exports in JSON format");
+ print_usage_line(0, "");
+
+ print_usage_line(1, "-export-dependencies-file:<filename>");
+ print_usage_line(2, "Specifies the filename for `-export-dependencies`.");
+ print_usage_line(2, "Example: -export-dependencies-file:dependencies.d");
+ print_usage_line(0, "");
+
print_usage_line(1, "-thread-count:<integer>");
print_usage_line(2, "Overrides the number of threads the compiler will use to compile with.");
print_usage_line(2, "Example: -thread-count:2");
@@ -1822,6 +2252,15 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(2, "Usage in code:");
print_usage_line(3, "#config(SPAM, default_value)");
print_usage_line(0, "");
+
+ print_usage_line(1, "-show-defineables");
+ print_usage_line(2, "Shows an overview of all the #config/#defined usages in the project.");
+ print_usage_line(0, "");
+
+ print_usage_line(1, "-export-defineables:<filename>");
+ print_usage_line(2, "Exports an overview of all the #config/#defined usages in CSV format to the given file path.");
+ print_usage_line(2, "Example: -export-defineables:defineables.csv");
+ print_usage_line(0, "");
}
if (build) {
@@ -1831,6 +2270,8 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(3, "-build-mode:exe Builds as an executable.");
print_usage_line(3, "-build-mode:dll Builds as a dynamically linked library.");
print_usage_line(3, "-build-mode:shared Builds as a dynamically linked library.");
+ print_usage_line(3, "-build-mode:lib Builds as a statically linked library.");
+ print_usage_line(3, "-build-mode:static Builds as a statically linked library.");
print_usage_line(3, "-build-mode:obj Builds as an object file.");
print_usage_line(3, "-build-mode:object Builds as an object file.");
print_usage_line(3, "-build-mode:assembly Builds as an assembly file.");
@@ -1877,9 +2318,9 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(0, "");
print_usage_line(1, "-use-separate-modules");
- print_usage_line(1, "[EXPERIMENTAL]");
print_usage_line(2, "The backend generates multiple build units which are then linked together.");
print_usage_line(2, "Normally, a single build unit is generated for a standard project.");
+ print_usage_line(2, "This is the default behaviour on Windows for '-o:none' and '-o:minimal' builds.");
print_usage_line(0, "");
}
@@ -1936,9 +2377,26 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(1, "-vet-semicolon");
print_usage_line(2, "Errs on unneeded semicolons.");
print_usage_line(0, "");
+
+ print_usage_line(1, "-vet-cast");
+ print_usage_line(2, "Errs on casting a value to its own type or using `transmute` rather than `cast`.");
+ print_usage_line(0, "");
+
+ print_usage_line(1, "-vet-tabs");
+ print_usage_line(2, "Errs when the use of tabs has not been used for indentation.");
+ print_usage_line(0, "");
}
if (check) {
+ print_usage_line(1, "-custom-attribute:<string>");
+ print_usage_line(2, "Add a custom attribute which will be ignored if it is unknown.");
+ print_usage_line(2, "This can be used with metaprogramming tools.");
+ print_usage_line(2, "Examples:");
+ print_usage_line(3, "-custom-attribute:my_tag");
+ print_usage_line(3, "-custom-attribute:my_tag,the_other_thing");
+ print_usage_line(3, "-custom-attribute:my_tag -custom-attribute:the_other_thing");
+ print_usage_line(0, "");
+
print_usage_line(1, "-ignore-unknown-attributes");
print_usage_line(2, "Ignores unknown attributes.");
print_usage_line(2, "This can be used with metaprogramming tools.");
@@ -1952,10 +2410,6 @@ gb_internal void print_show_help(String const arg0, String const &command) {
}
if (test_only) {
- print_usage_line(1, "-test-name:<string>");
- print_usage_line(2, "Runs specific test only by name.");
- print_usage_line(0, "");
-
print_usage_line(1, "-all-packages");
print_usage_line(2, "Tests all packages imported into the given initial package.");
print_usage_line(0, "");
@@ -1981,7 +2435,20 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(2, "Examples:");
print_usage_line(3, "-microarch:sandybridge");
print_usage_line(3, "-microarch:native");
- print_usage_line(3, "-microarch:? for a list");
+ print_usage_line(3, "-microarch:\"?\" for a list");
+ print_usage_line(0, "");
+
+ print_usage_line(1, "-target-features:<string>");
+ print_usage_line(2, "Specifies CPU features to enable on top of the enabled features implied by -microarch.");
+ print_usage_line(2, "Examples:");
+ print_usage_line(3, "-target-features:atomics");
+ print_usage_line(3, "-target-features:\"sse2,aes\"");
+ print_usage_line(3, "-target-features:\"?\" for a list");
+ print_usage_line(0, "");
+
+ print_usage_line(1, "-strict-target-features");
+ print_usage_line(2, "Makes @(enable_target_features=\"...\") behave the same way as @(require_target_features=\"...\").");
+ print_usage_line(2, "This enforces that all generated code uses features supported by the combination of -target, -microarch, and -target-features.");
print_usage_line(0, "");
print_usage_line(1, "-reloc-mode:<string>");
@@ -2002,6 +2469,12 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(0, "");
}
+ if (build) {
+ print_usage_line(1, "-print-linker-flags");
+ print_usage_line(2, "Prints the all of the flags/arguments that will be passed to the linker.");
+ print_usage_line(0, "");
+ }
+
if (check) {
print_usage_line(1, "-disallow-do");
print_usage_line(2, "Disallows the 'do' keyword in the project.");
@@ -2012,9 +2485,13 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(0, "");
print_usage_line(1, "-strict-style");
+ print_usage_line(2, "This enforces parts of same style as the Odin compiler, prefer '-vet-style -vet-semicolon' if you do not want to match it exactly.");
+ print_usage_line(2, "");
print_usage_line(2, "Errs on unneeded tokens, such as unneeded semicolons.");
print_usage_line(2, "Errs on missing trailing commas followed by a newline.");
print_usage_line(2, "Errs on deprecated syntax.");
+ print_usage_line(2, "Errs when the attached-brace style in not adhered to (also known as 1TBS).");
+ print_usage_line(2, "Errs when 'case' labels are not in the same column as the associated 'switch' token.");
print_usage_line(0, "");
print_usage_line(1, "-ignore-warnings");
@@ -2084,6 +2561,7 @@ gb_internal void print_show_help(String const arg0, String const &command) {
print_usage_line(2, "[Windows only]");
print_usage_line(2, "Defines the resource file for the executable.");
print_usage_line(2, "Example: -resource:path/to/file.rc");
+ print_usage_line(2, "or: -resource:path/to/file.res for a precompiled one.");
print_usage_line(0, "");
print_usage_line(1, "-pdb-name:<filepath>");
@@ -2466,7 +2944,6 @@ int main(int arg_count, char const **arg_ptr) {
TIME_SECTION("init args");
map_init(&build_context.defined_values);
build_context.extra_packages.allocator = heap_allocator();
- string_set_init(&build_context.test_names);
Array<String> args = setup_args(arg_count, arg_ptr);
@@ -2590,6 +3067,8 @@ int main(int arg_count, char const **arg_ptr) {
} else if (command == "root") {
gb_printf("%.*s", LIT(odin_root_dir()));
return 0;
+ } else if (command == "clear-cache") {
+ return try_clear_cache() ? 0 : 1;
} else {
String argv1 = {};
if (args.count > 1) {
@@ -2634,6 +3113,10 @@ int main(int arg_count, char const **arg_ptr) {
gb_printf_err("Expected either a directory or a .odin file, got '%.*s'\n", LIT(init_filename));
return 1;
}
+ if (!gb_file_exists(cast(const char*)init_filename.text)) {
+ gb_printf_err("The file '%.*s' was not found.\n", LIT(init_filename));
+ return 1;
+ }
}
}
}
@@ -2663,7 +3146,7 @@ int main(int arg_count, char const **arg_ptr) {
// Check chosen microarchitecture. If not found or ?, print list.
bool print_microarch_list = true;
- if (build_context.microarch.len == 0) {
+ if (build_context.microarch.len == 0 || build_context.microarch == str_lit("native")) {
// Autodetect, no need to print list.
print_microarch_list = false;
} else {
@@ -2680,6 +3163,11 @@ int main(int arg_count, char const **arg_ptr) {
}
}
+ // Set and check build paths...
+ if (!init_build_paths(init_filename)) {
+ return 1;
+ }
+
String default_march = get_default_microarchitecture();
if (print_microarch_list) {
if (build_context.microarch != "?") {
@@ -2703,13 +3191,57 @@ int main(int arg_count, char const **arg_ptr) {
return 0;
}
- // Set and check build paths...
- if (!init_build_paths(init_filename)) {
- return 1;
+ String march = get_final_microarchitecture();
+ String default_features = get_default_features();
+ {
+ String_Iterator it = {default_features, 0};
+ for (;;) {
+ String str = string_split_iterator(&it, ',');
+ if (str == "") break;
+ string_set_add(&build_context.target_features_set, str);
+ }
+ }
+
+ if (build_context.target_features_string.len != 0) {
+ String_Iterator target_it = {build_context.target_features_string, 0};
+ for (;;) {
+ String item = string_split_iterator(&target_it, ',');
+ if (item == "") break;
+
+ String invalid;
+ if (!check_target_feature_is_valid_for_target_arch(item, &invalid) && item != str_lit("help")) {
+ if (item != str_lit("?")) {
+ gb_printf_err("Unkown target feature '%.*s'.\n", LIT(invalid));
+ }
+ gb_printf("Possible -target-features for target %.*s are:\n", LIT(target_arch_names[build_context.metrics.arch]));
+ gb_printf("\n");
+
+ String feature_list = target_features_list[build_context.metrics.arch];
+ String_Iterator it = {feature_list, 0};
+ for (;;) {
+ String str = string_split_iterator(&it, ',');
+ if (str == "") break;
+ if (check_single_target_feature_is_valid(default_features, str)) {
+ if (has_ansi_terminal_colours()) {
+ gb_printf("\t%.*s\x1b[38;5;244m (implied by target microarch %.*s)\x1b[0m\n", LIT(str), LIT(march));
+ } else {
+ gb_printf("\t%.*s (implied by current microarch %.*s)\n", LIT(str), LIT(march));
+ }
+ } else {
+ gb_printf("\t%.*s\n", LIT(str));
+ }
+ }
+
+ return 1;
+ }
+
+ string_set_add(&build_context.target_features_set, item);
+ }
}
if (build_context.show_debug_messages) {
- debugf("Selected microarch: %.*s\n", LIT(default_march));
+ debugf("Selected microarch: %.*s\n", LIT(march));
+ debugf("Default microarch features: %.*s\n", LIT(default_features));
for_array(i, build_context.build_paths) {
String build_path = path_to_string(heap_allocator(), build_context.build_paths[i]);
debugf("build_paths[%ld]: %.*s\n", i, LIT(build_path));
@@ -2737,7 +3269,8 @@ int main(int arg_count, char const **arg_ptr) {
// TODO(jeroen): Remove the `init_filename` param.
// Let's put that on `build_context.build_paths[0]` instead.
if (parse_packages(parser, init_filename) != ParseFile_None) {
- return 1;
+ GB_ASSERT_MSG(any_errors(), "parse_packages failed but no error was reported.");
+ // We depend on the next conditional block to return 1, after printing errors.
}
if (any_errors()) {
@@ -2748,13 +3281,21 @@ int main(int arg_count, char const **arg_ptr) {
print_all_errors();
}
- MAIN_TIME_SECTION("type check");
checker->parser = parser;
init_checker(checker);
- defer (destroy_checker(checker));
+ defer (destroy_checker(checker)); // this is here because of a `goto`
+ if (build_context.cached && parser->total_seen_load_directive_count.load() == 0) {
+ MAIN_TIME_SECTION("check cached build (pre-semantic check)");
+ if (try_cached_build(checker, args)) {
+ goto end_of_code_gen;
+ }
+ }
+
+ MAIN_TIME_SECTION("type check");
check_parsed_files(checker);
+ check_defines(&build_context, checker);
if (any_errors()) {
print_all_errors();
return 1;
@@ -2763,6 +3304,19 @@ int main(int arg_count, char const **arg_ptr) {
print_all_errors();
}
+ if (build_context.show_defineables || build_context.export_defineables_file != "") {
+ TEMPORARY_ALLOCATOR_GUARD();
+ temp_alloc_defineable_strings(checker);
+ sort_defineables(checker);
+
+ if (build_context.show_defineables) {
+ show_defineables(checker);
+ }
+
+ if (build_context.export_defineables_file != "") {
+ export_defineables(checker, build_context.export_defineables_file);
+ }
+ }
if (build_context.command_kind == Command_strip_semicolon) {
return strip_semicolons(parser);
@@ -2792,6 +3346,13 @@ int main(int arg_count, char const **arg_ptr) {
return 0;
}
+ if (build_context.cached) {
+ MAIN_TIME_SECTION("check cached build");
+ if (try_cached_build(checker, args)) {
+ goto end_of_code_gen;
+ }
+ }
+
#if ALLOW_TILDE
if (build_context.tilde_backend) {
LinkerData linker_data = {};
@@ -2802,12 +3363,17 @@ int main(int arg_count, char const **arg_ptr) {
switch (build_context.build_mode) {
case BuildMode_Executable:
+ case BuildMode_StaticLibrary:
case BuildMode_DynamicLibrary:
i32 result = linker_stage(&linker_data);
if (result) {
if (build_context.show_timings) {
show_timings(checker, &global_timings);
}
+
+ if (build_context.export_dependencies_format != DependenciesExportUnspecified) {
+ export_dependencies(checker);
+ }
return result;
}
break;
@@ -2815,20 +3381,30 @@ int main(int arg_count, char const **arg_ptr) {
} else
#endif
{
- MAIN_TIME_SECTION("LLVM API Code Gen");
lbGenerator *gen = gb_alloc_item(permanent_allocator(), lbGenerator);
if (!lb_init_generator(gen, checker)) {
return 1;
}
+
+ gbString label_code_gen = gb_string_make(heap_allocator(), "LLVM API Code Gen");
+ if (gen->modules.count > 1) {
+ label_code_gen = gb_string_append_fmt(label_code_gen, " ( %4td modules )", gen->modules.count);
+ }
+ MAIN_TIME_SECTION_WITH_LEN(label_code_gen, gb_string_length(label_code_gen));
if (lb_generate_code(gen)) {
switch (build_context.build_mode) {
case BuildMode_Executable:
+ case BuildMode_StaticLibrary:
case BuildMode_DynamicLibrary:
i32 result = linker_stage(gen);
if (result) {
if (build_context.show_timings) {
show_timings(checker, &global_timings);
}
+
+ if (build_context.export_dependencies_format != DependenciesExportUnspecified) {
+ export_dependencies(checker);
+ }
return result;
}
break;
@@ -2838,15 +3414,27 @@ int main(int arg_count, char const **arg_ptr) {
remove_temp_files(gen);
}
+end_of_code_gen:;
+
if (build_context.show_timings) {
show_timings(checker, &global_timings);
}
+ if (build_context.export_dependencies_format != DependenciesExportUnspecified) {
+ export_dependencies(checker);
+ }
+
+
+ if (!build_context.build_cache_data.copy_already_done &&
+ build_context.cached) {
+ try_copy_executable_to_cache();
+ }
+
if (run_output) {
String exe_name = path_to_string(heap_allocator(), build_context.build_paths[BuildPath_Output]);
defer (gb_free(heap_allocator(), exe_name.text));
- return system_exec_command_line_app("odin run", "\"%.*s\" %.*s", LIT(exe_name), LIT(run_args_string));
+ system_must_exec_command_line_app("odin run", "\"%.*s\" %.*s", LIT(exe_name), LIT(run_args_string));
}
return 0;
}