Re: [PoC] Improve dead tuple storage for lazy vacuum

Hi,

On March 6, 2024 9:06:50 AM GMT+01:00, John Naylor <johncnaylorls(at)gmail(dot)com> wrote:
>On Wed, Mar 6, 2024 at 3:02 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
>>
>> On Wed, Mar 6, 2024 at 4:41 PM Andres Freund <andres(at)anarazel(dot)de> wrote:
>
>> > A few ARM buildfarm animals are complaining:
>> >
>> > https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=parula&dt=2024-03-06%2007%3A34%3A02
>> > https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=snakefly&dt=2024-03-06%2007%3A34%3A03
>> > https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=massasauga&dt=2024-03-06%2007%3A33%3A18
>> >
>>
>> The error message we got is:
>>
>> ../../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.
>
>That sounds plausible, and I'll look further.
>
>(Hmm, I thought we had run this code on Arm already...)

Perhaps we should switch one of the CI jobs to ARM...

Andres

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.