[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 00:05:27 PST 2022
NimishMishra created this revision.
NimishMishra added projects: Flang, OpenMP.
Herald added subscribers: jdoerfert, guansong, yaxunl.
Herald added a project: All.
NimishMishra requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a project: LLVM.
This NFC fixes formatting issues introduced with https://reviews.llvm.org/D110714
Repository:
rG LLVM Github Monorepo
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,7 +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>;
@@ -1367,8 +1368,8 @@
!(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 =
@@ -1394,7 +1395,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());
}
}
@@ -1417,7 +1419,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;
}
}
@@ -1432,8 +1435,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;
}
}
@@ -1442,8 +1446,8 @@
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.413715.patch
Type: text/x-patch
Size: 3195 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220308/dcd433fa/attachment.bin>
More information about the llvm-commits
mailing list