This revision was automatically updated to reflect the committed changes.
yaxunl marked an inline comment as done.
Closed by commit rG3f4b5893efed: [AMDGPU] Add option -munsafe-fp-atomics
(authored by yaxunl).
Herald added a project: clang.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAS
yaxunl marked an inline comment as done.
yaxunl added inline comments.
Comment at: clang/include/clang/Basic/TargetOptions.h:78
+ /// \brief If enabled, allow AMDGPU unsafe floating point atomics.
+ bool AllowAMDGPUUnsafeFPAtomics = false;
tra wrote:
> I'm cu
tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.
LGTM.
Patch description could use a pointer to more details about the unsafe atomics.
Comment at: clang/include/clang/Basic/TargetOptions.h:78
+ /// \brief If enabled, allow AM
yaxunl created this revision.
yaxunl added reviewers: tra, rjmccall, arsenm.
Herald added subscribers: dang, kerbowa, jfb, t-tye, tpr, dstuttard, nhaehnle,
jvesely, kzhuravl.
yaxunl requested review of this revision.
Herald added a subscriber: wdng.
Add an option -munsafe-fp-atomics for AMDGPU ta
