Date: Wed, 3 Sep 2025 09:48:57 -0500
From: Nathan Bossart <nathandbossart@gmail.com>
To: "Chiranmoy.Bhattacharya@fujitsu.com" <Chiranmoy.Bhattacharya@fujitsu.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, David Rowley <dgrowleyml@gmail.com>,
	John Naylor <johncnaylorls@gmail.com>,
	"pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>,
	"Devanga.Susmitha@fujitsu.com" <Devanga.Susmitha@fujitsu.com>,
	"Ragesh.Hajela@fujitsu.com" <Ragesh.Hajela@fujitsu.com>
Subject: Re: [PATCH] Hex-coding optimizations using SVE on ARM.
Message-ID: <aLhVWTRy0QPbW2tl@nathan>
References: <Z4qQMS8EtYzN-eno@nathan>
 <OSBPR01MB2664A733AFB69AB1A7A43CC297E12@OSBPR01MB2664.jpnprd01.prod.outlook.com>
 <OSBPR01MB266437C48E1F5ACB3D8C69C197E12@OSBPR01MB2664.jpnprd01.prod.outlook.com>
 <Z5VguuGPybo5ai_S@nathan>
 <TY2PR01MB26671195A71369185F7BD08C97F42@TY2PR01MB2667.jpnprd01.prod.outlook.com>
 <OSBPR01MB2664ED032A81FBF21BEC82EE97C52@OSBPR01MB2664.jpnprd01.prod.outlook.com>
 <aEBOVMM2TrXJ948v@nathan>
 <OS9PR01MB15185A8BA8E15E4AFD8B14490976BA@OS9PR01MB15185.jpnprd01.prod.outlook.com>
 <OS9PR01MB15185626CA7C6EAF82C93ABA3974FA@OS9PR01MB15185.jpnprd01.prod.outlook.com>
 <TY4PR01MB15177710A7DF163A8536F26289701A@TY4PR01MB15177.jpnprd01.prod.outlook.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: 
 <TY4PR01MB15177710A7DF163A8536F26289701A@TY4PR01MB15177.jpnprd01.prod.outlook.com>
Archived-At: <https://www.postgresql.org/message-id/aLhVWTRy0QPbW2tl%40nathan>
Precedence: bulk

On Wed, Sep 03, 2025 at 11:11:24AM +0000, Chiranmoy.Bhattacharya@fujitsu.com wrote:
> Since the CommitFest is underway, could we get some feedback to improve
> the patch?

I see that there was some discussion about a Neon implementation upthread,
but I'm not sure we concluded anything.  For popcount, we first added a
Neon version before adding the SVE version, which required more complicated
configure/runtime checks.  Presumably Neon is available on more hardware
than SVE, so that could be a good place to start here, too.

Also, I'd strongly encourage you to get involved with others' patches on
the mailing lists (e.g., reviewing, testing).  Patch submissions are great,
but this community depends on other types of participation, too.  IME
helping others with their patches also tends to incentivize others to help
with yours.

-- 
nathan