Re: use ARM intrinsics in pg_lfind32() where available
Nathan Bossart <nathandbossart@gmail.com>
From: Nathan Bossart <nathandbossart@gmail.com>
To: Andres Freund <andres@anarazel.de>
Cc: pgsql-hackers@postgresql.org, john.naylor@enterprisedb.com
Date: 2022-08-19T22:28:14Z
Lists: pgsql-hackers
Attachments
On Fri, Aug 19, 2022 at 02:26:02PM -0700, Andres Freund wrote: > Are you sure there's not an appropriate define for us to use here instead of a > configure test? E.g. > > echo|cc -dM -P -E -|grep -iE 'arm|aarch' > ... > #define __AARCH64_SIMD__ 1 > ... > #define __ARM_NEON 1 > #define __ARM_NEON_FP 0xE > #define __ARM_NEON__ 1 > .. > > I strikes me as non-scalable to explicitly test all the simd instructions we'd > use. Thanks for the pointer. GCC, Clang, and the Arm compiler all seem to define __ARM_NEON, so here is a patch that uses that instead. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
Commits
-
Further code review of port/simd.h
- 865424627db6 16.0 landed
-
Fix broken cast on MSVC
- c6a43c25a8ba 16.0 landed
-
Remove unused symbol __aarch64
- 4112e39f70ee 16.0 landed