[flang] [llvm] [flang][OpenMP] Overhaul implementation of ATOMIC construct (PR #137852)
Tom Eccles via llvm-commits
llvm-commits at lists.llvm.org
Wed May 28 10:19:06 PDT 2025
https://github.com/tblah commented:
Thanks @kparzysz for the massive effort this patch must have taken to implement.
I have a few minor comments but I agree with your overall approach.
Right now I cannot build this with -Werror:
```
[1m/home/$USER/llvm-project/flang/lib/Semantics/check-omp-structure.cpp:2820:56: [0m[0;1;31merror: [0m[1mmissing field 'iff' initializer [-Werror,-Wmissing-field-initializers][0m
2820 | GetActionStmt(std::get<parser::Block>(s.t))};[0m
| [0;1;32m ^
[0m1 error generated.
ninja: build stopped: subcommand failed.
```
https://github.com/llvm/llvm-project/pull/137852
More information about the llvm-commits
mailing list