From 5657608b5e9f9c47f2d95f30fd28e1967a33cb05 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A1n=20Tomko?= Date: Sat, 14 Dec 2019 22:10:15 +0100 Subject: [PATCH] build: warn on a large frame by default MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit My commit e73889b6311f5b43d859caa4bae84bfdb299967a split the -Wframe-larger-than warning setting into two different variables - STRICT_FRAME_LIMIT_CFLAGS for the library code and RELAXED_FRAME_LIMIT_CFLAGS which was needed for tests. Use the strict limit by default and specify the warning flag twice for the parts that require a larger stack frame, relying on the fact that the compiler will pick up the latter value. Signed-off-by: Ján Tomko Reviewed-by: Daniel Henrique Barboza --- m4/virt-compile-warnings.m4 | 2 +- src/Makefile.am | 2 -- tools/Makefile.am | 2 -- 3 files changed, 1 insertion(+), 5 deletions(-) diff --git a/m4/virt-compile-warnings.m4 b/m4/virt-compile-warnings.m4 index be5dc3de51..5f930756b0 100644 --- a/m4/virt-compile-warnings.m4 +++ b/m4/virt-compile-warnings.m4 @@ -171,7 +171,7 @@ AC_DEFUN([LIBVIRT_COMPILE_WARNINGS],[ # This should be < 256 really. Currently we're down to 4096, # but using 1024 bytes sized buffers (mostly for virStrerror) # stops us from going down further - gl_WARN_ADD([-Wframe-larger-than=4096], [STRICT_FRAME_LIMIT_CFLAGS]) + wantwarn="$wantwarn -Wframe-larger-than=4096" gl_WARN_ADD([-Wframe-larger-than=32768], [RELAXED_FRAME_LIMIT_CFLAGS]) # Extra special flags diff --git a/src/Makefile.am b/src/Makefile.am index 013976a2b7..cd01796d67 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -31,8 +31,6 @@ AM_CPPFLAGS = -I../gnulib/lib \ -Dabs_top_srcdir="\"$(abs_top_srcdir)\"" \ $(NULL) -WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS) - AM_CFLAGS = $(LIBXML_CFLAGS) \ $(GLIB_CFLAGS) \ $(WARN_CFLAGS) \ diff --git a/tools/Makefile.am b/tools/Makefile.am index 67b342d306..426a6ba661 100644 --- a/tools/Makefile.am +++ b/tools/Makefile.am @@ -29,8 +29,6 @@ AM_CPPFLAGS = \ # needed STANDALONE_CPPFLAGS = -I$(top_srcdir) -WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS) - AM_CFLAGS = \ $(WARN_CFLAGS) \ $(COVERAGE_CFLAGS) \