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

Reply via email to