Skip to content

Commit a07f512

Browse files
authored
Always use -Wl,--warn-unresolved-symbols in MSAN builds (#48835)
* Always use `-Wl,--warn-unresolved-symbols` in MSAN builds * Use `-Wl,--warn-unresolved-symbols` only on Linux and FreeBSD
1 parent fb86768 commit a07f512

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

Make.inc

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -674,7 +674,10 @@ SANITIZE_LDFLAGS :=
674674
ifeq ($(SANITIZE_MEMORY),1)
675675
SANITIZE_OPTS += -fsanitize=memory -fsanitize-memory-track-origins -fno-omit-frame-pointer
676676
SANITIZE_LDFLAGS += $(SANITIZE_OPTS)
677-
endif
677+
ifneq ($(findstring $(OS),Linux FreeBSD),)
678+
SANITIZE_LDFLAGS += -Wl,--warn-unresolved-symbols
679+
endif # OS Linux or FreeBSD
680+
endif # SANITIZE_MEMORY=1
678681
ifeq ($(SANITIZE_ADDRESS),1)
679682
SANITIZE_OPTS += -fsanitize=address
680683
SANITIZE_LDFLAGS += -fsanitize=address

deps/libsuitesparse.mk

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ LIBSUITESPARSE_MFLAGS := CC="$(CC) $(SANITIZE_OPTS)" CXX="$(CXX) $(SANITIZE_OPTS
2626
AR="$(AR)" RANLIB="$(RANLIB)" \
2727
BLAS="-L$(build_shlibdir) -lblastrampoline" \
2828
LAPACK="-L$(build_shlibdir) -lblastrampoline" \
29-
LDFLAGS="$(SUITESPARSE_LIB) $(SANITIZE_LDFLAGS) -Wl,--warn-unresolved-symbols" CFOPENMP="" CUDA=no CUDA_PATH="" \
29+
LDFLAGS="$(SUITESPARSE_LIB) $(SANITIZE_LDFLAGS)" CFOPENMP="" CUDA=no CUDA_PATH="" \
3030
UMFPACK_CONFIG="$(UMFPACK_CONFIG)" \
3131
CHOLMOD_CONFIG="$(CHOLMOD_CONFIG)" \
3232
SPQR_CONFIG="$(SPQR_CONFIG)"

0 commit comments

Comments
 (0)