RE: [PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread VandeVondele Joost
> a) I don't like the option name, -Womp-no-default-clause would be IMHO better a bit related to b), I first had -Womp-no-default-none-clause but it becomes long and contains twice 'no' (also consider the no-omp-no-default... form). If you like that more, I'll make that change (see below) other

Re: [PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 02:16:58PM +, 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