On Mon, May 11, 2026 at 5:42 PM Bruce Momjian <[email protected]> wrote: > On Mon, May 11, 2026 at 12:40:38PM +0700, John Naylor wrote: > > On Wed, Apr 15, 2026 at 8:19 AM Bruce Momjian <[email protected]> wrote: > > > > Hi Bruce, thanks for you work on this. Here are some comments: > > <snip> > > > Optionally use AVX2 CPU instructions for calculating page checksums > > > (Matthew Sterrett, Andrew Kim) § > > > > > Optionally use ARM Crypto Extension to Compute CRC32C (John Naylor) § > > > > There's no option for these, the extensions are used where available. > > So, the "Optionally" indicates we use these instructions if the CPU > supports them. I assume we don't use them on all CPUs. Can you > suggest better wording? >
I think the simplest change would be to replace "optionally" with "when available", although ISTR we use wording along the lines of "Add support for" or "is now supported" when runtime/compiler checks are involved. Robert Treat https://xzilla.net
