diff mbox series

[meta-oe,v2] nodejs: fix compile failed for 32 bit arm target

Message ID 20251028111956.2903925-1-hongxu.jia@windriver.com
State New
Headers show
Series [meta-oe,v2] nodejs: fix compile failed for 32 bit arm target | expand

Commit Message

Hongxu Jia Oct. 28, 2025, 11:19 a.m. UTC
Build failed for 32 bit arm target

$ echo 'MACHINE = "qemuarm"' >> conf/local.conf
$ bitbake nodejs
...
 2645 |         );
      |         ^
../deps/llhttp/src/llhttp.c:2643:11: error: incompatible type for argument 1 of 'vandq_u16'
 2643 |           vcgeq_u8(input, vdupq_n_u8(' ')),
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |           |
      |           uint8x16_t
...

Use '-flax-vector-conversions' to permit conversions between vectors
with differing element types or numbers of subparts

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Khem Raj Oct. 29, 2025, 1:38 a.m. UTC | #1
I wonder if it needs to enable neon machine feature, I am not seeing
this failure with clang.

On Tue, Oct 28, 2025 at 4:20 AM hongxu via lists.openembedded.org
<hongxu.jia=eng.windriver.com@lists.openembedded.org> wrote:
>
> Build failed for 32 bit arm target
>
> $ echo 'MACHINE = "qemuarm"' >> conf/local.conf
> $ bitbake nodejs
> ...
>  2645 |         );
>       |         ^
> ../deps/llhttp/src/llhttp.c:2643:11: error: incompatible type for argument 1 of 'vandq_u16'
>  2643 |           vcgeq_u8(input, vdupq_n_u8(' ')),
>       |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>       |           |
>       |           uint8x16_t
> ...
>
> Use '-flax-vector-conversions' to permit conversions between vectors
> with differing element types or numbers of subparts
>
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>  meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb b/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
> index 6d1a364be0..7a784bf7f2 100644
> --- a/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
> +++ b/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
> @@ -48,6 +48,10 @@ CVE_PRODUCT += "node.js"
>  # v8 errors out if you have set CCACHE
>  CCACHE = ""
>
> +# Use '-flax-vector-conversions' to permit conversions between vectors
> +# with differing element types or numbers of subparts
> +CFLAGS:append:arm = " -flax-vector-conversions"
> +
>  def map_nodejs_arch(a, d):
>      import re
>
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#121077): https://lists.openembedded.org/g/openembedded-devel/message/121077
> Mute This Topic: https://lists.openembedded.org/mt/115991441/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Hongxu Jia Oct. 30, 2025, 5:11 a.m. UTC | #2
On 10/29/25 09:38, Khem Raj via lists.openembedded.org wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> I wonder if it needs to enable neon machine feature, I am not seeing
> this failure with clang.

Agree, clang toolchain does not need it, I will apply the option for gcc 
toolchain only

//Hongxu

>
> On Tue, Oct 28, 2025 at 4:20 AM hongxu via lists.openembedded.org
> <hongxu.jia=eng.windriver.com@lists.openembedded.org> wrote:
>> Build failed for 32 bit arm target
>>
>> $ echo 'MACHINE = "qemuarm"' >> conf/local.conf
>> $ bitbake nodejs
>> ...
>>   2645 |         );
>>        |         ^
>> ../deps/llhttp/src/llhttp.c:2643:11: error: incompatible type for argument 1 of 'vandq_u16'
>>   2643 |           vcgeq_u8(input, vdupq_n_u8(' ')),
>>        |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>        |           |
>>        |           uint8x16_t
>> ...
>>
>> Use '-flax-vector-conversions' to permit conversions between vectors
>> with differing element types or numbers of subparts
>>
>> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>> ---
>>   meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb b/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
>> index 6d1a364be0..7a784bf7f2 100644
>> --- a/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
>> +++ b/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
>> @@ -48,6 +48,10 @@ CVE_PRODUCT += "node.js"
>>   # v8 errors out if you have set CCACHE
>>   CCACHE = ""
>>
>> +# Use '-flax-vector-conversions' to permit conversions between vectors
>> +# with differing element types or numbers of subparts
>> +CFLAGS:append:arm = " -flax-vector-conversions"
>> +
>>   def map_nodejs_arch(a, d):
>>       import re
>>
>> --
>> 2.34.1
>>
>>
>>
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#121139): https://lists.openembedded.org/g/openembedded-devel/message/121139
>> Mute This Topic: https://lists.openembedded.org/mt/115991441/3617049
>> Group Owner: openembedded-devel+owner@lists.openembedded.org
>> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [hongxu.jia@eng.windriver.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
diff mbox series

Patch

diff --git a/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb b/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
index 6d1a364be0..7a784bf7f2 100644
--- a/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
+++ b/meta-oe/recipes-devtools/nodejs/nodejs_22.21.0.bb
@@ -48,6 +48,10 @@  CVE_PRODUCT += "node.js"
 # v8 errors out if you have set CCACHE
 CCACHE = ""
 
+# Use '-flax-vector-conversions' to permit conversions between vectors
+# with differing element types or numbers of subparts
+CFLAGS:append:arm = " -flax-vector-conversions"
+
 def map_nodejs_arch(a, d):
     import re