Re: Improve CRC32C performance on SSE4.2

From: Nathan Bossart <nathandbossart(at)gmail(dot)com>
To: Tomas Vondra <tomas(at)vondra(dot)me>
Cc: Andy Fan <zhihuifan1213(at)163(dot)com>, John Naylor <johncnaylorls(at)gmail(dot)com>, "Devulapalli, Raghuveer" <raghuveer(dot)devulapalli(at)intel(dot)com>, "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-14 13:56:34
Message-ID: aE1_knL40G9Wq_zp@nathan
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Jun 14, 2025 at 03:47:33PM +0200, Tomas Vondra wrote:
> I suggest you try with a newer gcc, perhaps 13.4. There's been a bunch
> of fixes related to AVX512 since 13.0, chances are this was already
> fixed. I don't see this failure on 14.3.1.

From the config.log, it looks like Andy is using clang:

configure:3998: clang --version >&5
clang version 18.1.6 (https://gitee.com/mirrors/llvm-project.git 1118c2e05e67a36ed8ca250524525cdb66a55256)

And I see -O0 used, too, which would match the existing report [0],
although that report is for clang 19.1.7.

I'm also genuinely curious why folks are using -O0...

[0] https://postgr.es/m/CAE-ML%2B-OV6p9uvCFBcSQjZUEh__y0h-KjN%2BBseyGJHt7u8EP%2Bw%40mail.gmail.com

--
nathan

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2025-06-14 14:01:38 Re: Improve CRC32C performance on SSE4.2
Previous Message Tomas Vondra 2025-06-14 13:50:56 Re: Handling OID Changes in Regression Tests for C Extensions