Re: Improve CRC32C performance on SSE4.2

From: John Naylor <johncnaylorls(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andy Fan <zhihuifan1213(at)163(dot)com>, "Devulapalli, Raghuveer" <raghuveer(dot)devulapalli(at)intel(dot)com>, Jesper Pedersen <jesperpedersen(dot)db(at)gmail(dot)com>, Tomas Vondra <tomas(at)vondra(dot)me>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "Shankaran, Akash" <akash(dot)shankaran(at)intel(dot)com>
Subject: Re: Improve CRC32C performance on SSE4.2
Date: 2025-06-24 03:34:12
Message-ID: CANWCAZbMwQbk+cWPWWk5vzHhWLNw774c7cbDG0X394JX+oF6sA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jun 23, 2025 at 10:05 PM Nathan Bossart
<nathandbossart(at)gmail(dot)com> wrote:
>
> On Mon, Jun 23, 2025 at 10:51:21AM -0400, Tom Lane wrote:
> > This has broken the build completely on my RHEL8 x86_64 box,
> > with gcc 8.5.0:
> >
> > $ ./configure ...
> > $ make -s
> > pg_crc32c_sse42.c: In function 'pg_comp_crc32c_avx512':
> > pg_crc32c_sse42.c:126:25: warning: implicit declaration of function '_mm512_zextsi128_si512'; did you mean '_mm512_castsi128_si512'? [-Wimplicit-function-declaration]
> > x0 = _mm512_xor_si512(_mm512_zextsi128_si512(_mm_cvtsi32_si128(crc0)), x0);
> > ^~~~~~~~~~~~~~~~~~~~~~
> > _mm512_castsi128_si512
>
> It looks like thse weren't added until GCC 10 [0].

Huh, that's surprising because the Intel manual put it in AVX-512F,
the basic core around which everything else is tacked on.

--
John Naylor
Amazon Web Services

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message jian he 2025-06-24 03:34:56 Re: split func.sgml to separated individual sgml files
Previous Message Yugo Nagata 2025-06-24 03:19:48 Re: Per-role disabling of LEAKPROOF requirements for row-level security?