Messages in this thread |  | | From | Kees Cook <> | Date | Mon, 13 Aug 2018 13:10:52 -0700 | Subject | Re: [PATCH] gcc-plugins: require GCC |
| |
On Mon, Aug 13, 2018 at 12:38 AM, Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > 2018-08-11 18:48 GMT+09:00 Stefan Agner <stefan@agner.ch>: >> Unsurprisingly GCC plugins require GCC as a compiler. This avoids >> GCC plugins being selectable when using clang. >> >> Signed-off-by: Stefan Agner <stefan@agner.ch> >> --- >> arch/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/Kconfig b/arch/Kconfig >> index 1aa59063f1fd..8c693a837ed7 100644 >> --- a/arch/Kconfig >> +++ b/arch/Kconfig >> @@ -422,7 +422,7 @@ config HAVE_GCC_PLUGINS >> >> menuconfig GCC_PLUGINS >> bool "GCC plugins" >> - depends on HAVE_GCC_PLUGINS >> + depends on HAVE_GCC_PLUGINS && CC_IS_GCC >> depends on PLUGIN_HOSTCC != "" >> help >> GCC plugins are loadable modules that provide extra features to the >> -- >> 2.18.0 >> > > I guess the more correct way is to fix > scripts/gcc-plugin.sh > > This shell script should exit 0 > only when GCC plugin is supported.
I'm of two minds: we already have the gcc test in Kconfig so we might want to avoid redundant checks, but maybe the script should be a "complete" test. I guess the latter. I will send a patch.
-Kees
-- Kees Cook Pixel Security
|  |