[libvirt] [PATCHv2] Split out -Wframe-larger-than warning from WARN_CLFAGS

Introduce STRICT_FRAME_LIMIT_CFLAGS that will be used for the library code and RELAXED_FRAME_LIMIT_CFLAGS for daemon code and the test code. Raising the limit for tests allows building them with clang with optimizations disabled. --- v2: * use the strict limit for tools and daemon too * append the limit to WARN_CFLAGS once per Makefile daemon/Makefile.am | 2 ++ m4/virt-compile-warnings.m4 | 4 ++-- src/Makefile.am | 2 ++ tests/Makefile.am | 2 ++ tools/Makefile.am | 2 ++ 5 files changed, 10 insertions(+), 2 deletions(-) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index 60c7368..d02ab33 100644 --- a/daemon/Makefile.am +++ b/daemon/Makefile.am @@ -31,6 +31,8 @@ INCLUDES = \ CLEANFILES = +WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS) + DAEMON_GENERATED = \ remote_dispatch.h \ lxc_dispatch.h \ diff --git a/m4/virt-compile-warnings.m4 b/m4/virt-compile-warnings.m4 index 17fdf9d..768a5c8 100644 --- a/m4/virt-compile-warnings.m4 +++ b/m4/virt-compile-warnings.m4 @@ -169,8 +169,8 @@ 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 - wantwarn="$wantwarn -Wframe-larger-than=4096" - dnl wantwarn="$wantwarn -Wframe-larger-than=256" + gl_WARN_ADD(["-Wframe-larger-than=4096"], [STRICT_FRAME_LIMIT_CFLAGS]) + gl_WARN_ADD(["-Wframe-larger-than=25600"], [RELAXED_FRAME_LIMIT_CFLAGS]) # Extra special flags dnl -fstack-protector stuff passes gl_WARN_ADD with gcc diff --git a/src/Makefile.am b/src/Makefile.am index 75e4344..3c7adae 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -37,6 +37,8 @@ INCLUDES = -I../gnulib/lib \ -Dabs_topsrcdir="\"$(abs_topsrcdir)\"" \ $(GETTEXT_CPPFLAGS) +WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS) + AM_CFLAGS = $(LIBXML_CFLAGS) \ $(WARN_CFLAGS) \ $(LOCK_CHECKING_CFLAGS) \ diff --git a/tests/Makefile.am b/tests/Makefile.am index aa9d2eb..279e9b7 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -33,6 +33,8 @@ INCLUDES = \ -I$(top_srcdir)/src/conf \ $(GETTEXT_CPPFLAGS) +WARN_CFLAGS += $(RELAXED_FRAME_LIMIT_CFLAGS) + AM_CFLAGS = \ -Dabs_builddir="\"$(abs_builddir)\"" \ -Dabs_topbuilddir="\"$(abs_topbuilddir)\"" \ diff --git a/tools/Makefile.am b/tools/Makefile.am index 162d8e5..bfacaf2 100644 --- a/tools/Makefile.am +++ b/tools/Makefile.am @@ -23,6 +23,8 @@ INCLUDES = \ -I$(top_srcdir) \ $(GETTEXT_CPPFLAGS) +WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS) + AM_LDFLAGS = \ $(RELRO_LDFLAGS) \ $(NO_INDIRECT_LDFLAGS) \ -- 2.10.2