[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