Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider using __atomic* builtins instead of __sync* ones: the first support 64-bit atomics on all platforms #80

Open
barracuda156 opened this issue Sep 9, 2024 · 0 comments

Comments

@barracuda156
Copy link

src/tcpkali_atomic.h uses old __sync* built-ins, which only support 32-bit atomics on 32-bit platforms, and then we get:

#warning \
    "This compiler does not have 64-bit compare_and_swap, results might be broken"

Instead, if the code will be changed to use __atomic* built-ins (which is what gcc upstream recommends), 64-bit atomics will be available on all platforms (32-bit ones may need explicit linking to libatomic).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant