[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