[flang-commits] [flang] [Flang][OpenMP] Allow copyprivate and nowait on the directive clauses (PR #127769)
Kelvin Li via flang-commits
flang-commits at lists.llvm.org
Wed Feb 26 12:02:31 PST 2025
kkwli wrote:
> > > Would it make sense to allow this usage irrespective of the standard?
> >
> >
> > My two cents. If flang supports both forms, it'd better to disallow mixing two forms in a **single** construct. There is if **copyprivate** is specified on the **single** directive, there shouldn't be anything on the **end single**. I guess that is less confusing.
>
> I would argue that this should be warning instead of a hard error. The code would not be incorrect according to the OpenMP API and therefore should be accepted, and the warning should tell the user to be consistent.
I fine with lowering to a warning.
https://github.com/llvm/llvm-project/pull/127769
More information about the flang-commits
mailing list