[PATCH] D71726: Let clang atomic builtins fetch add/sub support floating point types

James Y Knight via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 21 15:05:10 PDT 2020

Previous message: [PATCH] D83955: [PowerPC][Power10] Implementation of 128-bit Binary Vector Multiply builtins
Next message: [PATCH] D71726: Let clang atomic builtins fetch add/sub support floating point types
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

jyknight added a comment.

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?


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

https://reviews.llvm.org/D71726

Previous message: [PATCH] D83955: [PowerPC][Power10] Implementation of 128-bit Binary Vector Multiply builtins
Next message: [PATCH] D71726: Let clang atomic builtins fetch add/sub support floating point types
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the cfe-commits mailing list