Message234030
I commited atomicv5.patch because it's simple, but I'm not sure that we are using stdatomic.h "correctly". The current code looks to be written for GCC, Clang fails to compile it (FreeBSD 10 now uses Clang instead of GCC).
Maybe the "_Atomic void*" type is wrong, and we should use the "atomic_uintptr_t" type instead and cast to the expected type? Attached atomic_pointer.patch implements this idea. It works on FreeBSD 10 with Clang and on Fedora 21 with GCC 4.9, both have stdatomic.h. |
|
Date |
User |
Action |
Args |
2015-01-14 15:06:22 | vstinner | set | recipients:
+ vstinner, Arfrever, neologix, python-dev, koobs, John.Malmberg, Vitor.de.Lima, gustavotemple |
2015-01-14 15:06:22 | vstinner | set | messageid: <1421247982.26.0.357122886149.issue22038@psf.upfronthosting.co.za> |
2015-01-14 15:06:22 | vstinner | link | issue22038 messages |
2015-01-14 15:06:22 | vstinner | create | |
|