On 12/16/2015 03:53 AM, Pierre-Marie de Rodat wrote:
+  /* Called from finalize_size_functions for functions whose body is needed to
+     generate complete debug info.  For instance, functions used to compute the
+     size of variable-length structures.  */
+  void (* function_body) (tree decl);

Calling this "function_body" seems overly generic; let's call it size_function and talk specifically about encoding the function body in the debug info.

   debug_nothing_rtx_insn,               /* var_location */
+  debug_nothing_tree,                   /* var_location */

And this comment shouldn't be the same as the previous line.

+/* Helper for loc_descr_without_nops: free the location description operation
+   P.  */
+bool
+free_loc_descr (const dw_loc_descr_ref &loc, void *data ATTRIBUTE_UNUSED)

Blank line between comment and function.

OK with those changes.

Jason

Reply via email to