| From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
|---|---|
| To: | John Naylor <johncnaylorls(at)gmail(dot)com> |
| Cc: | Ants Aasma <ants(at)cybertec(dot)at>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: add AVX2 support to simd.h |
| Date: | 2024-01-02 16:11:23 |
| Message-ID: | 20240102161123.GA955987@nathanxps13 |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Jan 01, 2024 at 07:12:26PM +0700, John Naylor wrote:
> On Thu, Nov 30, 2023 at 12:15 AM Nathan Bossart
> <nathandbossart(at)gmail(dot)com> wrote:
>> I don't intend for this patch to be
>> seriously considered until we have better support for detecting/compiling
>> AVX2 instructions and a buildfarm machine that uses them.
>
> That's completely understandable, yet I'm confused why there is a
> commitfest entry for it marked "needs review".
Perhaps I was too optimistic about adding support for newer instructions...
I'm tempted to propose that we move forward with this patch as-is after
adding a buildfarm machine that compiles with -mavx2 or -march=x86-64-v3.
There is likely still follow-up work to make these improvements more
accessible, but I'm not sure that is a strict prerequisite here.
(In case it isn't clear, I'm volunteering to set up such a buildfarm
machine.)
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2024-01-02 16:20:54 | Re: introduce dynamic shared memory registry |
| Previous Message | Robert Haas | 2024-01-02 15:34:11 | Re: trying again to get incremental backup |