llvm: add NDEBUG when assertion mode is off

`llvm-config --cxxflags` unfortunately does not set `-DNDEBUG`, which Julia needs to set correctly when including LLVM header files.
JuliaLang · May 24, 2022 · c9c2082 · c9c2082
1 parent fc52b3f
commit c9c2082
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/Makefile b/src/Makefile
@@ -110,6 +110,11 @@ PUBLIC_HEADER_TARGETS := $(addprefix $(build_includedir)/julia/,$(notdir $(PUBLI
 LLVM_LDFLAGS := $(shell $(LLVM_CONFIG_HOST) --ldflags)
 LLVM_CXXFLAGS := $(shell $(LLVM_CONFIG_HOST) --cxxflags)
 
+# llvm-config --cxxflags does not return -DNDEBUG
+ifeq ($(shell $(LLVM_CONFIG_HOST) --assertion-mode),OFF)
+LLVM_CXXFLAGS += -DNDEBUG
+endif
+
 ifeq ($(JULIACODEGEN),LLVM)
 ifneq ($(USE_SYSTEM_LLVM),0)
 CG_LLVMLINK += $(LLVM_LDFLAGS) $(shell $(LLVM_CONFIG_HOST) --libs --system-libs)