[flang-commits] [flang] ed2fa6f - Revert "[flang][OpenMP] Fix unused variable warning (#153409)"
Krzysztof Parzyszek via flang-commits
flang-commits at lists.llvm.org
Wed Aug 13 08:28:01 PDT 2025
Author: Krzysztof Parzyszek
Date: 2025-08-13T10:27:53-05:00
New Revision: ed2fa6fd89df5ac28645d05854710e5b5d62dc4b
URL: https://github.com/llvm/llvm-project/commit/ed2fa6fd89df5ac28645d05854710e5b5d62dc4b
DIFF: https://github.com/llvm/llvm-project/commit/ed2fa6fd89df5ac28645d05854710e5b5d62dc4b.diff
LOG: Revert "[flang][OpenMP] Fix unused variable warning (#153409)"
This reverts commit 6b20b16b2f796b01bc5c3d5d8013a8e65b40080a.
This patch depends on PR153098 which is the one being reverted.
Added:
Modified:
flang/lib/Semantics/check-omp-atomic.cpp
Removed:
################################################################################
diff --git a/flang/lib/Semantics/check-omp-atomic.cpp b/flang/lib/Semantics/check-omp-atomic.cpp
index 2e16d42a2a99a..f98a00070f742 100644
--- a/flang/lib/Semantics/check-omp-atomic.cpp
+++ b/flang/lib/Semantics/check-omp-atomic.cpp
@@ -718,7 +718,7 @@ OmpStructureChecker::CheckUpdateCapture(
void OmpStructureChecker::CheckAtomicCaptureAssignment(
const evaluate::Assignment &capture, const SomeExpr &atom,
parser::CharBlock source) {
- auto [_, rsrc]{SplitAssignmentSource(source)};
+ auto [lsrc, rsrc]{SplitAssignmentSource(source)};
const SomeExpr &cap{capture.lhs};
if (!IsVarOrFunctionRef(atom)) {
@@ -734,7 +734,7 @@ void OmpStructureChecker::CheckAtomicCaptureAssignment(
void OmpStructureChecker::CheckAtomicReadAssignment(
const evaluate::Assignment &read, parser::CharBlock source) {
- auto [_, rsrc]{SplitAssignmentSource(source)};
+ auto [lsrc, rsrc]{SplitAssignmentSource(source)};
if (auto maybe{GetConvertInput(read.rhs)}) {
const SomeExpr &atom{*maybe};
@@ -820,7 +820,7 @@ OmpStructureChecker::CheckAtomicUpdateAssignment(
std::pair<bool, bool> OmpStructureChecker::CheckAtomicUpdateAssignmentRhs(
const SomeExpr &atom, const SomeExpr &rhs, parser::CharBlock source,
bool suppressDiagnostics) {
- auto [_, rsrc]{SplitAssignmentSource(source)};
+ auto [lsrc, rsrc]{SplitAssignmentSource(source)};
std::pair<operation::Operator, std::vector<SomeExpr>> top{
operation::Operator::Unknown, {}};
More information about the flang-commits
mailing list