================
@@ -6571,11 +6571,14 @@ class AtomicExpr : public Expr {
/// \return empty atomic scope model if the atomic op code does not have
/// scope operand.
static std::unique_ptr<AtomicScopeModel> getScopeModel(AtomicOp Op) {
- if (Op >= AO__opencl_atomic_load && Op <= AO__opencl_atomic_fetch_max)
+ // FIXME: Allow grouping of builtins to be able to only check >= and <=
+ if (Op >= AO__opencl_atomic_compare_exchange_strong &&
+ Op <= AO__opencl_atomic_store && Op != AO__opencl_atomic_init)
----------------
AaronBallman wrote:
I think it's fine for the moment, the FIXME comment clarifies what needs to be
done to improve it in the future.
I would also be fine if you wanted to tackle it now (perhaps we should have
some marking in the .td file to say "these should be grouped together as a
range"?).
https://github.com/llvm/llvm-project/pull/68324
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
