[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