[flang-commits] [flang] [Flang][OpenMP] Allow copyprivate and nowait on the directive clauses (PR #127769)

Michael Klemm via flang-commits flang-commits at lists.llvm.org
Mon Feb 24 08:42:32 PST 2025


mjklemm 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.

https://github.com/llvm/llvm-project/pull/127769


More information about the flang-commits mailing list