Re: [PATCH 6/7] md/raid10, LLVM: get rid of variable length array

From: Fengguang Wu
Date: Fri Mar 17 2017 - 20:42:17 EST


Hi Dmitry,

On Fri, Mar 17, 2017 at 08:05:16PM +0100, Dmitry Vyukov wrote:
On Fri, Mar 17, 2017 at 7:57 PM, Borislav Petkov <bp@xxxxxxxxx> wrote:
On Fri, Mar 17, 2017 at 07:47:33PM +0100, Dmitry Vyukov wrote:
This problem is more general and is not specific to clang. It equally
applies to different versions of gcc, different arches and different
configs (namely, anything else than what a developer used for
testing).

I guess. We do carry a bunch of gcc workarounds along with the cc-*
macros in scripts/Kbuild.include.

A known, reasonably well working solution to this problem is
a system of try bots that test patches before commit with different
compilers/configs/archs. We already have such system in the form of
0-day bots. It would be useful to extend it with clang as soon as
kernel builds.

Has someone actually already talked to Fengguang about it?

+Fengguang

I've actually tried clang long time ago. It quickly fails the build
for vanilla kernel. So it really depends on when the various clang
build fix patches can be accepted into mainline kernel.

Thanks,
Fengguang