Skip to content

Commit

Permalink
kbuild: remove cc-option test of -fno-stack-check
Browse files Browse the repository at this point in the history
The minimal compiler version, GCC 4.9 supports this flag.

Nathan Chancellor pointed out:
 "This flag is technically ignored by clang (see commit
 05b0798916f01690b5903302e51f3136274e291f) but that obviously
 does not matter for the sake of this."

Here is the godbolt:
https://godbolt.org/z/59cK6o

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Reviewed-by: Nathan Chancellor <natechancellor@gmail.com>
  • Loading branch information
Masahiro Yamada committed Sep 24, 2020
1 parent 8b42cf2 commit 7d4eb0d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -940,7 +940,7 @@ KBUILD_CFLAGS += $(call cc-option,-fno-merge-all-constants)
KBUILD_CFLAGS += $(call cc-option,-fmerge-constants)

# Make sure -fstack-check isn't enabled (like gentoo apparently did)
KBUILD_CFLAGS += $(call cc-option,-fno-stack-check,)
KBUILD_CFLAGS += -fno-stack-check

# conserve stack if available
KBUILD_CFLAGS += $(call cc-option,-fconserve-stack)
Expand Down

0 comments on commit 7d4eb0d

Please sign in to comment.