File tree Expand file tree Collapse file tree 1 file changed +4
-5
lines changed Expand file tree Collapse file tree 1 file changed +4
-5
lines changed Original file line number Diff line number Diff line change @@ -3388,18 +3388,17 @@ bool Sema::CheckAMDGCNBuiltinFunctionCall(unsigned BuiltinID,
3388
3388
3389
3389
// Check valididty of memory ordering as per C11 / C++11's memody model.
3390
3390
// Only fence needs check. Atomic dec/inc allow all memory orders.
3391
- auto DiagInvalidMemOrder = [&](auto *ArgExpr) {
3391
+ if (!llvm::isValidAtomicOrderingCABI(Ord))
3392
3392
return Diag(ArgExpr->getBeginLoc(),
3393
3393
diag::warn_atomic_op_has_invalid_memory_order)
3394
3394
<< ArgExpr->getSourceRange();
3395
- };
3396
- if (!llvm::isValidAtomicOrderingCABI(Ord))
3397
- return DiagInvalidMemOrder(ArgExpr);
3398
3395
switch (static_cast<llvm::AtomicOrderingCABI>(Ord)) {
3399
3396
case llvm::AtomicOrderingCABI::relaxed:
3400
3397
case llvm::AtomicOrderingCABI::consume:
3401
3398
if (BuiltinID == AMDGPU::BI__builtin_amdgcn_fence)
3402
- return DiagInvalidMemOrder(ArgExpr);
3399
+ return Diag(ArgExpr->getBeginLoc(),
3400
+ diag::warn_atomic_op_has_invalid_memory_order)
3401
+ << ArgExpr->getSourceRange();
3403
3402
break;
3404
3403
case llvm::AtomicOrderingCABI::acquire:
3405
3404
case llvm::AtomicOrderingCABI::release:
You can’t perform that action at this time.
0 commit comments