On Mon, May 06, 2013 at 03:25:43PM -0400, Jason Merrill wrote: > On 05/06/2013 02:25 PM, Jakub Jelinek wrote: > >I did it that way because if I understand the code well, often > >grokdeclarator/start_decl/start_function/grokmethod/grokfield > >is called with a chain of declarators, not just a single one. > > True, perhaps adding it to declspecs->attributes would be simpler.
Not to declspecs->attributes, that is a tree, and we need to record vec<tree, gc> *, but would you be fine if it was cp_decl_specifier_seq field (omp_declare_simd_clauses in there)? Looking at the code, cp_decl_specifier_seq is just one unlike a chain of declarators, and technically #pragma omp declare simd is closer to another kind of decl specifier, something that comes before the declarator itself. Jakub