[flang-commits] [flang] [flang] Fix build breakage with FLANG_ENABLE_WERROR on (NFC) (PR #162894)
via flang-commits
flang-commits at lists.llvm.org
Fri Oct 10 10:33:32 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-fir-hlfir
Author: Kelvin Li (kkwli)
<details>
<summary>Changes</summary>
Fix the compile error:
```
llvm-project/flang/lib/Lower/OpenMP/OpenMP.cpp:3386:32: error: variable 'dir' set but not used [-Werror,-Wunused-but-set-variable]
3386 | switch (llvm::omp::Directive dir = item->id) {
| ^
1 error generated.
```
---
Full diff: https://github.com/llvm/llvm-project/pull/162894.diff
1 Files Affected:
- (modified) flang/lib/Lower/OpenMP/OpenMP.cpp (+2-1)
``````````diff
diff --git a/flang/lib/Lower/OpenMP/OpenMP.cpp b/flang/lib/Lower/OpenMP/OpenMP.cpp
index bd94651919960..444f27471020b 100644
--- a/flang/lib/Lower/OpenMP/OpenMP.cpp
+++ b/flang/lib/Lower/OpenMP/OpenMP.cpp
@@ -3383,7 +3383,8 @@ static void genOMPDispatch(lower::AbstractConverter &converter,
}
}
- switch (llvm::omp::Directive dir = item->id) {
+ llvm::omp::Directive dir = item->id;
+ switch (dir) {
case llvm::omp::Directive::OMPD_barrier:
newOp = genBarrierOp(converter, symTable, semaCtx, eval, loc, queue, item);
break;
``````````
</details>
https://github.com/llvm/llvm-project/pull/162894
More information about the flang-commits
mailing list