[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 21 15:20:12 PDT 2020


yaxunl added a comment.

In D71726#2165424 <https://reviews.llvm.org/D71726#2165424>, @jyknight wrote:

> Why not have clang always emit atomicrmw for floats, and let AtomicExpandPass handle legalizing that into integer atomics if necessary, rather than adding a target hook in clang?


Not all targets can legalize fp atomics by AtomicExpandPass. Some targets need library support.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71726/new/

https://reviews.llvm.org/D71726





More information about the cfe-commits mailing list