On Wed, Nov 18, 2015 at 11:02:01AM -0800, Cesar Philippidis wrote:
>  static inline void
>  cp_ensure_no_oacc_routine (cp_parser *parser)
>  {
> -  cp_finalize_oacc_routine (parser, NULL_TREE, false, true);
> +  if (parser->oacc_routine && !parser->oacc_routine->error_seen)
> +    {
> +      tree clauses = parser->oacc_routine->clauses;
> +      location_t loc = OMP_CLAUSE_LOCATION (TREE_PURPOSE(clauses));

Formatting, missing space before (clauses));

> @@ -19326,7 +19330,11 @@ cp_parser_late_return_type_opt (cp_parser* parser, 
> cp_declarator *declarator,
>      declarator->std_attributes
>        = cp_parser_late_parsing_omp_declare_simd (parser,
>                                                declarator->std_attributes);
> -
> +  if (oacc_routine_p)
> +    declarator->std_attributes
> +      = cp_parser_late_parsing_oacc_routine (parser,
> +                                          declarator->std_attributes);
> +  

Trailing whitespace at the end of line.

Otherwise LGTM.

        Jakub

Reply via email to