On Tue, Sep 23, 2014 at 02:16:58PM +0000, VandeVondele  Joost wrote:
> Attached patch introduces an optional warning for an OMP
> parallel/task/teams construct without explicit default(none).  This would
> effectively catch the number one reason of easy-to-avoid OMP bugs in our
> project.

a) I don't like the option name, -Womp-no-default-clause would be IMHO better
b) I think you shouldn't warn for explicit default(shared), or
   default(firstprivate) or default(private) clauses, there user explicitly
   tells what should happen for the non-listed vars (and it is pretty rare)
c) in the documentation I think you should make it clear that it is just a
   coding style warning (dunno if we have some verbiage for such warnings)

        Jakub

Reply via email to