On Tue, Jul 27, 2021 at 12:28:22PM +1000, Greg Nancarrow wrote:
> That patch didn't apply for me (on latest source) so I've attached an
> equivalent with those changes, that does apply, and also tweaks the
> Makefile include path to address that #include issue.

When applying some micro-benchmarking to stress those APIs, how much
does this change things?  At the end of the day, this also comes down
to an evaluation of pg_ulltoa_n() and pg_ultoa_n().

 #include "common/int.h"
+#include "d2s_intrinsics.h"
Er, are you sure about this part?  The first version of the patch did
that in a different, also incorrect, way.
--
Michael

Attachment: signature.asc
Description: PGP signature

Reply via email to