Skip to content

[flang] Fix possibly unused variable (NFC)#200905

Merged
kkwli merged 1 commit into
llvm:mainfrom
kkwli:fix-unused-var
Jun 2, 2026
Merged

[flang] Fix possibly unused variable (NFC)#200905
kkwli merged 1 commit into
llvm:mainfrom
kkwli:fix-unused-var

Conversation

@kkwli
Copy link
Copy Markdown
Contributor

@kkwli kkwli commented Jun 1, 2026

Build fails with -DFLANG_ENABLE_WERROR=ON.

@kkwli kkwli requested a review from chandraghale June 1, 2026 18:58
@kkwli kkwli self-assigned this Jun 1, 2026
@llvmorg-github-actions llvmorg-github-actions Bot added flang Flang issues not falling into any other category flang:fir-hlfir labels Jun 1, 2026
@llvmorg-github-actions
Copy link
Copy Markdown

@llvm/pr-subscribers-flang-fir-hlfir

Author: Kelvin Li (kkwli)

Changes

Build fails with -DFLANG_ENABLE_WERROR=ON.


Full diff: https://github.com/llvm/llvm-project/pull/200905.diff

1 Files Affected:

  • (modified) flang/lib/Lower/OpenMP/Atomic.cpp (+1-1)
diff --git a/flang/lib/Lower/OpenMP/Atomic.cpp b/flang/lib/Lower/OpenMP/Atomic.cpp
index b80564fddd943..9d711b92bb520 100644
--- a/flang/lib/Lower/OpenMP/Atomic.cpp
+++ b/flang/lib/Lower/OpenMP/Atomic.cpp
@@ -650,7 +650,7 @@ void Fortran::lower::omp::lowerAtomic(
     // writeActionCond is a bitmask combining the following flags:
     //  1) the action type (Read/Write/Update)
     //  2) condition (IfTrue/IfFalse)
-    int writeActionCond = 0;
+    [[maybe_unused]] int writeActionCond = 0;
     const evaluate::Assignment *writeAssign = nullptr;
     if (analysis.op0.what & analysis.Write) {
       writeAssign = get(analysis.op0.assign);

Copy link
Copy Markdown
Contributor

@chandraghale chandraghale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg !!

@kkwli kkwli merged commit 0697815 into llvm:main Jun 2, 2026
13 checks passed
@kkwli kkwli deleted the fix-unused-var branch June 2, 2026 14:46
@kkwli
Copy link
Copy Markdown
Contributor Author

kkwli commented Jun 2, 2026

Thanks.

yingopq pushed a commit to yingopq/llvm-project that referenced this pull request Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flang:fir-hlfir flang Flang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants