Hi Nathan! On Fri, 9 Oct 2015 09:34:51 -0400, Nathan Sidwell wrote: > On 10/09/15 09:26, Thomas Schwinge wrote: > > You mean the cp_parser_oacc_loop and cp_parser_oacc_kernels_parallel > > functions need documentation? I agree it's a bit terse, but documenting > > these by just listing the accepted parsing tokens "# pragma acc loop" > > etc., followed by the *_CLAUSE_MASKs is what's done for the other > > OpenACC/OpenMP directives in the C/C++ front ends. So, I don't see a > > reason to be different for these two? > > > What's the p_name argument for? It's s string describing the pragma as parsed thus far. Again, not documenting that as well as our usage of it is totally "standard", see OpenMP's cp_parser_omp_parallel, cp_parser_omp_for, and many more. Grüße, Thomas