-
Notifications
You must be signed in to change notification settings - Fork 258
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
[question] Clang confuse the atomic header file to include. #1338
Comments
And this is my CMakelists.txt:
build.gradle:
|
#1178 (comment) and down is the interesting part. This is a defect in the C++ standard. We'll probably put our workaround back into r22. It was fixed in r21 but the fix broke a lot of things (things that weren't right, but were "working") so we reverted since we didn't want the LTS release to be the one introducing the breaking change. |
Actually, this is already fixed in r22. |
@DanAlbert Could you tell how to work around this issue? |
If possible, don't include stdatomic.h before atomic in the same TU. If not... You could maybe copy the stdatomic.h from r21 into your project's include path. |
It seem that cmake use
./toolchains/llvm/prebuilt/linux-x86_64/lib64/clang/8.0.7/include/stdatomic.h
instead of./toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/stdatomic.h
, which cause errors.Error log:
And
The text was updated successfully, but these errors were encountered: