[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: svn commit: r1841078 - in /apr/apr/trunk: CHANGES apr.dsp atomic/unix/builtins64.c atomic/unix/mutex64.c atomic/win32/apr_atomic64.c include/apr_atomic.h include/arch/unix/apr_arch_atomic.h test/testatomic.c

Moved from httpd dev (which was moved to BCC)

On Sep 17, 2018, at 2:54 PM, Yann Ylavic <ylavic.dev@xxxxxxxxx> wrote:

On Mon, Sep 17, 2018 at 5:52 PM Jim Jagielski <jim@xxxxxxxxxxx> wrote:

Would like to also propose for apr-1.7...

How about 128bit? :p

There are __int128 (gcc) and _m128 (MSVC) and most 64bit intel/amd
CPUs support cmpxchg16b.

The ‘__atomic’ builtins can be used with any integral scalar or pointer type that is 1, 2, 4, or 8 bytes in length. 16-byte integral types are also allowed if ‘__int128’ (see __int128) is supported by the architecture.

This also implies that APR create a 128bit Int type, which we don't (yet).

Bumping to support 64bit was easy and logical.. bumping to 128 requires more legwork and is a bit more "intrusive" ;)

+1 to the theory though.