pitrou commented on code in PR #37785:
URL: https://github.com/apache/arrow/pull/37785#discussion_r1331788781
##########
go/parquet/internal/bmi/bitmap_neon_arm64.s:
##########
@@ -27,8 +27,8 @@ LBB0_2:
MOVD R0, res+16(FP)
RET
LBB0_4:
- WORD $0xaa1f03e0 // mov x0, xzr
WORD $0xa8c17bfd // ldp x29, x30, [sp], #16
+ MOVD ZR, res+16(FP)
Review Comment:
Do you think it's a C/C++ compiler bug? That sounds a bit unlikely to me,
given how simple the `extract_bits` function is.
Also, why are you compiling this code for Neon? I find it unlikely that Neon
can be used to speed up your SW emulation of `PEXT`.
Also, we have a SW emulation in Parquet C++ that processes 5 bits at a time
rather than one. You could perhaps try to compile that one (or rewrite it in
Go) :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
