[Bug 63566] atomic builtins detection not quite correct

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=63566

Ivan A. Melnikov <iv...@altlinux.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |iv@altlinux.org,
                   |                            |jim@apache.org

--- Comment #1 from Ivan A. Melnikov <iv...@altlinux.org> ---
I'm getting the same error on linux on my mipsel (32-bit LE) machines. On
32-bit MIPS gcc also does not have 64-bit built-in atomics. I'm currently
disabling nonportable atomics for our mips32 builds.

> apr should probably do the configure test with a known 8 byte type
> instead, to make the test match the actual usage of the condition.

Since there some provisioning for having special implementation for 32-bit
atomics and using generic code for 64-bit atomics, I think it's better to have
two separate checks -- for 32-bit and 64-bit builtins. I'd provide a patch but
I'm not sure what is the best way of creating uint64_t value in the check code
without including stdint.h.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org