[PATCH] D71726: Let clang atomic builtins fetch add/sub support floating point types
Yaxun Liu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 28 11:07:35 PDT 2020
yaxunl updated this revision to Diff 281296.
yaxunl added a comment.
Make IEEE single and double type as supported for fp atomics in all targets by default. This is based on the assumption that AtomicExpandPass or its ongoing work is sufficient to support fp atomics for all targets. This is to facilitate middle end and backend end development to support fp atomics.
If a target would like to treat single and double fp atomics as unsupported, it can override the default behavior in its own TargetInfo.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71726/new/
https://reviews.llvm.org/D71726
Files:
clang/include/clang/Basic/DiagnosticSemaKinds.td
clang/include/clang/Basic/TargetInfo.h
clang/lib/CodeGen/CGAtomic.cpp
clang/lib/Sema/SemaChecking.cpp
clang/test/CodeGenCUDA/amdgpu-atomic-ops.cu
clang/test/CodeGenOpenCL/atomic-ops.cl
clang/test/Sema/atomic-ops.c
clang/test/SemaCUDA/amdgpu-atomic-ops.cu
clang/test/SemaOpenCL/atomic-ops.cl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71726.281296.patch
Type: text/x-patch
Size: 15328 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200728/9e6fbe8d/attachment-0001.bin>
More information about the cfe-commits
mailing list