[llvm] [Frontend][OpenMP] Remove `reduction` from allowed clauses for `target` (PR #90754)
Alexey Bataev via llvm-commits
llvm-commits at lists.llvm.org
Wed May 1 11:19:27 PDT 2024
alexey-bataev wrote:
> There will be build failures because I haven't changed any tests.
>
> In commit [3f96fe6](https://github.com/llvm/llvm-project/commit/3f96fe6d44774c8f892ff9b678c9cfa1d6df1539), Alexey added support for `reduction` on `target`, seemingly in response to a change in OpenMP 5.0. The issue is that the 5.0 spec added support for `in_reduction`, not `reduction` to `target`. The `reduction` clause is not allowed to this day.
>
> Is this some kind of an extension we decided to implement?
I don't remember already, probably some kind of extension to support combined directives codegen (most probably).
https://github.com/llvm/llvm-project/pull/90754
More information about the llvm-commits
mailing list