From 589820639c38979f4c801e8edcbb62c21ca15099 Mon Sep 17 00:00:00 2001 From: gingerBill Date: Thu, 10 Aug 2023 14:35:09 +0100 Subject: [PATCH] Correct `lb_run_function_pass_manager` behaviour --- src/llvm_backend_opt.cpp | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/src/llvm_backend_opt.cpp b/src/llvm_backend_opt.cpp index 54e667a0b..068546e9d 100644 --- a/src/llvm_backend_opt.cpp +++ b/src/llvm_backend_opt.cpp @@ -374,12 +374,24 @@ gb_internal void lb_run_function_pass_manager(LLVMPassManagerRef fpm, lbProcedur if (p == nullptr) { return; } - LLVMRunFunctionPassManager(fpm, p->value); // NOTE(bill): LLVMAddDCEPass doesn't seem to be exported in the official DLL's for LLVM // which means we cannot rely upon it // This is also useful for read the .ll for debug purposes because a lot of instructions // are not removed lb_run_remove_dead_instruction_pass(p); + + switch (pass_manager_kind) { + case lbFunctionPassManager_none: + return; + case lbFunctionPassManager_default: + case lbFunctionPassManager_default_without_memcpy: + if (build_context.optimization_level < 0) { + return; + } + break; + } + + LLVMRunFunctionPassManager(fpm, p->value); } gb_internal void llvm_delete_function(LLVMValueRef func) {