]> granicus.if.org Git - postgresql/commit
Fix portability issues in pg_bitutils
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 13 Feb 2019 23:09:48 +0000 (20:09 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 13 Feb 2019 23:09:48 +0000 (20:09 -0300)
commit109de05cbb034b032cd60f50708716c8ff0afdf2
treed851e8295a7df4f306650cf763948a8149bacb78
parent80c468b4a454881b56e1c73c6fedcb2978c5b415
Fix portability issues in pg_bitutils

We were using uint64 function arguments as "long int" arguments to
compiler builtins, which fails on machines where long ints are 32 bits:
the upper half of the uint64 was being ignored.  Fix by using the "ll"
builtin variants instead, which on those machines take 64 bit arguments.

Also, remove configure tests for __builtin_popcountl() (as well as
"long" variants for ctz and clz): the theory here is that any compiler
version will provide all widths or none, so one test suffices.  Were
this theory to be wrong, we'd have to add tests for
__builtin_popcountll() and friends, which would be tedious.

Per failures in buildfarm member lapwing and ensuing discussion.
config/c-compiler.m4
configure
configure.in
src/include/pg_config.h.in
src/include/pg_config.h.win32
src/port/pg_bitutils.c