public inbox for [email protected]  
help / color / mirror / Atom feed
From: Andrew Kim <[email protected]>
To: [email protected]
Cc: [email protected]
Cc: Matthew Sterrett <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: Andrew Kim <[email protected]>
Cc: Andrew Kim <[email protected]>
Cc: [email protected]
Subject: Re: Proposal for enabling auto-vectorization for checksum calculations
Date: Tue, 16 Sep 2025 18:10:56 -0700
Message-ID: <CAK64mneJWJ4wsvzcDhP4XOdSgjxf=fba_OSpaF2_Uo5MmtXN=g@mail.gmail.com> (raw)

> I don't know if this is related to the crashes, but it doesn't seem
> like a good idea to #include the function pointer stuff everywhere,
> that should probably go into src/port like the others.
Just a gentle reminder on this patch series — I’ve already rebased it on
the latest tip of master and addressed the earlier review comments:

  * Moved the function pointer definitions into src/port as suggested.
  * Rebased cleanly on the current master branch.

Could someone take another look and share any further feedback?

Thanks a lot for your time and review,
Andrew Kim


view thread (35+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Proposal for enabling auto-vectorization for checksum calculations
  In-Reply-To: <CAK64mneJWJ4wsvzcDhP4XOdSgjxf=fba_OSpaF2_Uo5MmtXN=g@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox