Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint - Mailing list pgsql-hackers

From Chapman Flack
Subject Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint
Date
Msg-id 6179E5D5.5060102@anastigmatix.net
Whole thread Raw
In response to Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint  (Arjan van de Ven <arjan@linux.intel.com>)
Responses Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tree view 
On 10/27/21 18:18, Arjan van de Ven wrote:
> +        /*
> +         * Special case each of the possible base values  (8, 10, 16) to
> avoid an
> +         * expensive divide operation
> +         * (the compiler will use a multiply, shift or boolean ops for this)
> +         */


Was 'boolean' the intended word there? To me it is distinct from 'bitwise'.

Regards,
-Chap

pgsql-hackers by date:

Previous
From: Mark Dilger
Date:
Subject: Re: CREATEROLE and role ownership hierarchies
Next
From: Kyotaro Horiguchi
Date:
Subject: Re: Isn't it better with "autovacuum worker...." instead of "worker took too long to start; canceled" specific to "auto