Skip to content

Commit 23eeee0

Browse files
pks-tgitster
authored andcommitted
Makefile: drop -DSUPPRESS_ANNOTATED_LEAKS
The -DSUPPRESS_ANNOTATED_LEAKS preprocessor directive was used to enable our `UNLEAK()` macro in the past, which marks memory as still-reachable so that the leak sanitizer does not complain. Starting with 52c7dbd (git-compat-util: drop now-unused `UNLEAK()` macro, 2024-11-20) this macro has been removed, and thus the preprocessor directive is not required anymore, either. Drop it. Signed-off-by: Patrick Steinhardt <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 714c134 commit 23eeee0

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1490,7 +1490,6 @@ ifneq ($(filter undefined,$(SANITIZERS)),)
14901490
BASIC_CFLAGS += -DSHA1DC_FORCE_ALIGNED_ACCESS
14911491
endif
14921492
ifneq ($(filter leak,$(SANITIZERS)),)
1493-
BASIC_CFLAGS += -DSUPPRESS_ANNOTATED_LEAKS
14941493
BASIC_CFLAGS += -O0
14951494
SANITIZE_LEAK = YesCompiledWithIt
14961495
endif

meson.build

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -712,7 +712,6 @@ else
712712
build_options_config.set('SANITIZE_ADDRESS', '')
713713
endif
714714
if get_option('b_sanitize').contains('leak')
715-
libgit_c_args += '-DSUPPRESS_ANNOTATED_LEAKS'
716715
build_options_config.set('SANITIZE_LEAK', 'YesCompiledWithIt')
717716
else
718717
build_options_config.set('SANITIZE_LEAK', '')

0 commit comments

Comments
 (0)