On 07/01/2013 05:18 AM, Jakub Jelinek wrote:
        (cp_finish_omp_declare_simd): Renamed to...
        (cp_finalize_omp_declare_simd): ... this.  Adjust for
...
        (cp_finish_omp_declare_simd): New function.

So now we have "finish" and "finalize"; please choose more distinct names. Instead of "finish", perhaps cp_parser_late_parsing_omp_declare_simd to match other deferred parsing functions.

+  tree save_ccp = current_class_ptr;
+  tree save_ccr = current_class_ref;
+  if (quals >= 0)
+    inject_this_parameter (current_class_type, quals);

Let's share this code with the trailing-return-type rather than duplicate it. Perhaps by changing cp_parser_late_return_opt to handle both the trailing return type and omp declare simd.

        (omp_remove_redundant_declare_simd_attrs): New function.

This doesn't seem to be used anywhere.

Jason

Reply via email to