On Wed, Mar 6, 2024 at 5:33 PM John Naylor <johncnaylo...@gmail.com> wrote: > > On Wed, Mar 6, 2024 at 3:02 PM Masahiko Sawada <sawada.m...@gmail.com> wrote: > > > > ../../src/include/port/simd.h:326:71: error: incompatible type for > > argument 1 of \342\200\230vshrq_n_s8\342\200\231 > > uint8x16_t masked = vandq_u8(vld1q_u8(mask), (uint8x16_t) vshrq_n_s8(v, > > 7)); > > ^ > > > > Since 'v' is uint8x16_t I think we should have used vshrq_n_u8() instead. > > I've looked around and it seems clang is more lax on conversions. > Since it works fine for clang, I think we just need a cast here for > gcc. I've attached a blind attempt at a fix -- I'll apply shortly > unless someone happens to test and find it doesn't work.
I've reproduced the same error on my raspberry pi, and confirmed the patch fixes the error. My previous idea was wrong. With my proposal, the regression test for radix tree failed on my raspberry pi. On the other hand, with your patch the tests passed. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
