diff options
| author | gingerBill <gingerBill@users.noreply.github.com> | 2018-10-24 09:40:05 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-10-24 09:40:05 +0100 |
| commit | 2bd85e764e32beba498addad93aef6f618395394 (patch) | |
| tree | 1b65f3cdf2a043761019c0acb9e8adcab73c5a70 /src/build_settings.cpp | |
| parent | 822e4894f2a37d372dcb46db5185ad2c4d140d98 (diff) | |
| parent | d894fb37087a100eeea2a0de1da89bda390e1306 (diff) | |
Merge pull request #260 from lachsinc/master
[WIP] Provide llvm with more debug info (for Visual Studio debugger support)
Diffstat (limited to 'src/build_settings.cpp')
| -rw-r--r-- | src/build_settings.cpp | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/src/build_settings.cpp b/src/build_settings.cpp index 90a80c158..ec2f27343 100644 --- a/src/build_settings.cpp +++ b/src/build_settings.cpp @@ -589,10 +589,23 @@ void init_build_context(void) { bc->optimization_level = gb_clamp(bc->optimization_level, 0, 3); - gbString opt_flags = gb_string_make_reserve(heap_allocator(), 16); + gbString opt_flags = gb_string_make_reserve(heap_allocator(), 64); + opt_flags = gb_string_append_fmt(opt_flags, "-O%d ", bc->optimization_level); if (bc->optimization_level != 0) { - opt_flags = gb_string_append_fmt(opt_flags, "-O%d", bc->optimization_level); + // NOTE(lachsinc): The following options were previously passed during call + // to opt in main.cpp:exec_llvm_opt(). + // -die: Dead instruction elimination + // -memcpyopt: MemCpy optimization + opt_flags = gb_string_appendc(opt_flags, "-memcpyopt -die "); } + + // NOTE(lachsinc): This optimization option was previously required to get + // around an issue in fmt.odin. Thank bp for tracking it down! Leaving for now until the issue + // is resolved and confirmed by Bill. Maybe it should be readded in non-debug builds. + // if (bc->ODIN_DEBUG == false) { + // opt_flags = gb_string_appendc(opt_flags, "-mem2reg "); + // } + bc->opt_flags = make_string_c(opt_flags); |