Re: [Buildroot] [PATCH] package/wpewebkit: fix NEON build with gcc-12

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Michael,

On Mon,  1 May 2023 14:32:03 +0200
Michael Nosthoff via buildroot <buildroot@buildroot.org> wrote:

> since gcc-12 gcc complains about attributes in between decl-specifiers.
> This code is NEON specific so it should not appear on builds for other systems.
> 
> Signed-off-by: Michael Nosthoff <buildroot@heine.tech>

Could you provide in the commit log a defconfig that exhibits the
issue, since there's apparently no autobuilder failure for this issue.

> diff --git a/package/wpewebkit/0002-change-position-of-alignas.patch b/package/wpewebkit/0002-change-position-of-alignas.patch
> new file mode 100644
> index 0000000000..5a1564c4c1
> --- /dev/null
> +++ b/package/wpewebkit/0002-change-position-of-alignas.patch
> @@ -0,0 +1,33 @@
> +From e2a3e36c8df59d83de0a48b806cad5bb5515a658 Mon Sep 17 00:00:00 2001
> +From: Michael Nosthoff <michael.nosthoff@iba-group.com>
> +Date: Tue, 18 Apr 2023 19:47:03 +0200
> +Subject: [PATCH] change position of alignas
> +
> +fixes:
> +Source/WebCore/platform/graphics/cpu/arm/filters/FELightingNEON.cpp:38:8: error: standard attributes in middle of decl-specifiers
> +   38 | static alignas(16) short s_FELightingConstantsForNeon[] = {
> +      |        ^~~~~~~
> +
> +on gcc-12
> +
> +Signed-off-by: Michael Nosthoff <buildroot@heine.tech>

We need an Upstream: tag in new patches, so it really needs to be
submitted upstream.

Adrian, could you provide some guidance here?

I guess the ultimate upstream is https://github.com/WebKit/WebKit/ and
then changes in WebKit at some point get propagated into WPEWebkit?

Could you submit a pull request to upstream WebKit ?

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot