Skip to content

Commit 90bae4d

Browse files
nickdesaulniersmpe
authored andcommitted
powerpc/xmon: Reapply "Relax frame size for clang"
This is a manual revert of commit 7f3c5d0 ("Revert "powerpc/xmon: Relax frame size for clang"") but using ccflags-$(CONFIG_CC_IS_CLANG) which is shorter. Turns out that this is reproducible still under specific compiler versions (mea culpa: I did not test every supported version of clang), and even a few randconfigs bots found. We'll have to revisit this again in the future, for now back this out. Reported-by: Nathan Chancellor <[email protected]> Closes: ClangBuiltLinux#252 (comment) Reported-by: kernel test robot <[email protected]> Closes: https://lore.kernel.org/llvm/[email protected]/ Suggested-by: Nathan Chancellor <[email protected]> Reviewed-by: Nathan Chancellor <[email protected]> Signed-off-by: Nick Desaulniers <[email protected]> Signed-off-by: Michael Ellerman <[email protected]> Link: https://msgid.link/[email protected]
1 parent 4c33bf1 commit 90bae4d

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

arch/powerpc/xmon/Makefile

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@ ccflags-remove-$(CONFIG_FUNCTION_TRACER) += $(CC_FLAGS_FTRACE)
1212

1313
ccflags-$(CONFIG_PPC64) := $(NO_MINIMAL_TOC)
1414

15+
# Clang stores addresses on the stack causing the frame size to blow
16+
# out. See https://github.com/ClangBuiltLinux/linux/issues/252
17+
ccflags-$(CONFIG_CC_IS_CLANG) += -Wframe-larger-than=4096
18+
1519
obj-y += xmon.o nonstdio.o spr_access.o xmon_bpts.o
1620

1721
ifdef CONFIG_XMON_DISASSEMBLY

0 commit comments

Comments
 (0)