[PATCH] D121186: [NFC][flang][OpenMP] Fixes formatting issues with D110714

Nimish Mishra via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 8 05:49:29 PST 2022


NimishMishra updated this revision to Diff 413779.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121186

Files:
  flang/lib/Semantics/check-omp-structure.cpp


Index: flang/lib/Semantics/check-omp-structure.cpp
===================================================================
--- flang/lib/Semantics/check-omp-structure.cpp
+++ flang/lib/Semantics/check-omp-structure.cpp
@@ -1344,8 +1344,8 @@
     if ((exprLeft.value().source.ToString() != variableName) &&
         (exprRight.value().source.ToString() != variableName)) {
       context_.Say(variable.GetSource(),
-          "Atomic update variable '%s' not found in the RHS of the \
-assignment statement in an ATOMIC (UPDATE) construct"_err_en_US,
+          "Atomic update variable '%s' not found in the RHS of the "
+          "assignment statement in an ATOMIC (UPDATE) construct"_err_en_US,
           variableName);
     }
     return common::HasMember<T, AllowedBinaryOperators>;
@@ -1368,8 +1368,9 @@
                 !(name->source == "max" || name->source == "min" ||
                     name->source == "iand" || name->source == "ior" ||
                     name->source == "ieor")) {
-              context_.Say(expr.source, "Invalid intrinsic procedure name in \
-OpenMP ATOMIC (UPDATE) statement"_err_en_US);
+              context_.Say(expr.source,
+                  "Invalid intrinsic procedure name in "
+                  "OpenMP ATOMIC (UPDATE) statement"_err_en_US);
             } else if (name) {
               bool foundMatch{false};
               if (auto varDesignatorIndirection =
@@ -1395,8 +1396,8 @@
               }
               if (!foundMatch) {
                 context_.Say(expr.source,
-                    "Atomic update variable '%s' not found in the \
-argument list of intrinsic procedure"_err_en_US,
+                    "Atomic update variable '%s' not found in the "
+                    "argument list of intrinsic procedure"_err_en_US,
                     var.GetSource().ToString());
               }
             }
@@ -1419,8 +1420,8 @@
       numMemoryOrderClause++;
       if (numMemoryOrderClause > 1) {
         context_.Say(clause.source,
-            "More than one memory order clause not allowed on OpenMP \
-Atomic construct"_err_en_US);
+            "More than one memory order clause not allowed on OpenMP "
+            "Atomic construct"_err_en_US);
         return;
       }
     }
@@ -1435,9 +1436,9 @@
     if (std::get_if<Fortran::parser::OmpMemoryOrderClause>(&clause.u)) {
       numMemoryOrderClause++;
       if (numMemoryOrderClause > 1) {
-        context_.Say(
-            clause.source, "More than one memory order clause not allowed on \
-OpenMP Atomic construct"_err_en_US);
+        context_.Say(clause.source,
+            "More than one memory order clause not allowed on "
+            "OpenMP Atomic construct"_err_en_US);
         return;
       }
     }
@@ -1446,8 +1447,9 @@
     if (std::get_if<Fortran::parser::OmpMemoryOrderClause>(&clause.u)) {
       numMemoryOrderClause++;
       if (numMemoryOrderClause > 1) {
-        context_.Say(clause.source, "More than one memory order clause not \
-allowed on OpenMP Atomic construct"_err_en_US);
+        context_.Say(clause.source,
+            "More than one memory order clause not "
+            "allowed on OpenMP Atomic construct"_err_en_US);
         return;
       }
     }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121186.413779.patch
Type: text/x-patch
Size: 3219 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220308/871c2e46/attachment.bin>


More information about the llvm-commits mailing list