diff options
author | Stewart Smith <stewart@linux.ibm.com> | 2019-03-29 10:11:00 +1100 |
---|---|---|
committer | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2020-03-27 11:49:49 +0530 |
commit | df77c744be496008b3d83c977b6d40ae6268442f (patch) | |
tree | e5592fd8152975284a1b802b0cc622d846b72e11 | |
parent | f4e234a8dfbc8f6599a022dfd1243ebbc970a200 (diff) | |
download | skiboot-df77c744be496008b3d83c977b6d40ae6268442f.zip skiboot-df77c744be496008b3d83c977b6d40ae6268442f.tar.gz skiboot-df77c744be496008b3d83c977b6d40ae6268442f.tar.bz2 |
Bump allowed stack frame size for unit tests/host programs
[ Upstream commit 11374f20e8f1c179571077292dd2b3011dea4ae2 ]
We tend to have a lot more things inlined when building unit tests,
so let's just up the -Wframe-larger-than to avoid hitting it.
This time, it was noticed in travis-ci with the ubuntu:latest
image.
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
-rw-r--r-- | Makefile.main | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/Makefile.main b/Makefile.main index a155e1e..0202c55 100644 --- a/Makefile.main +++ b/Makefile.main @@ -30,7 +30,6 @@ CWARNS := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \ -Wmissing-prototypes -Wmissing-declarations \ -Wwrite-strings -Wcast-align \ -Winit-self \ - -Wframe-larger-than=1024 \ -Wno-stringop-truncation \ -Werror @@ -45,6 +44,12 @@ HOSTCFLAGS += $(call try-cflag,$(HOSTCC),-Wjump-misses-init) \ $(call try-cflag,$(HOSTCC),-Wsuggest-attribute=noreturn) HOSTCFLAGS += -DDEBUG -DCCAN_LIST_DEBUG +# We want small stack usage for skiboot +# but host compilation of unit tests tend to inline heavily, +# which creates larger stack frames and triggering useless warnings +HOSTCFLAGS += -Wframe-larger-than=4096 +CWARNS += -Wframe-larger-than=1024 + HOSTGCOVCFLAGS = -fprofile-arcs -ftest-coverage -lgcov -O0 -g -pg VALGRIND := valgrind -q --show-reachable=yes --error-exitcode=99 |