diff options
| author | Stewart Smith <stewart@linux.ibm.com> | 2019-03-29 10:11:00 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.ibm.com> | 2019-03-29 10:11:00 +1100 |
| commit | 11374f20e8f1c179571077292dd2b3011dea4ae2 (patch) | |
| tree | ae69d613d44c68f05e0b4391a16df686b4784215 | |
| parent | 133cc8cea2582f79b324c4de60ef36152a2087b3 (diff) | |
| download | talos-skiboot-11374f20e8f1c179571077292dd2b3011dea4ae2.tar.gz talos-skiboot-11374f20e8f1c179571077292dd2b3011dea4ae2.zip | |
Bump allowed stack frame size for unit tests/host programs
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>
| -rw-r--r-- | Makefile.main | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/Makefile.main b/Makefile.main index a8e34d16..8a99ec9f 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 \ $(call try-cflag,$(CC),--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 |
