[PATCH] D116637: [Clang][Sema][OpenMP] Sema support for `atomic compare`

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 3 10:23:49 PST 2022


ABataev added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:11027
+  if (auto *CS = dyn_cast<CompoundStmt>(Then)) {
+    if (CS->size() == 0) {
+      ErrorInfo.Error = ErrorTy::NoStmt;
----------------
`body_empty()`


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:11206
+    if (!E->isInstantiationDependent()) {
+      auto Type = E->getType();
+      if (!Type->isScalarType()) {
----------------
`QualType`


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:11241
+  if (CS) {
+    if (CS->size() == 0) {
+      ErrorInfo.Error = ErrorTy::NoStmt;
----------------
`body_empty()`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116637



More information about the cfe-commits mailing list