Re: [PATCH][ubsan] Add VLA bound instrumentation

[Jason Merrill]

On 10/30/2013 10:52 AM, Marek Polacek wrote:
+         if ((flag_sanitize & SANITIZE_VLA)
+             && !processing_template_decl

You don't need to check processing_template_decl; the template case was 
already handled above.

+             tree x = cp_save_expr (size);
+             x = build2 (COMPOUND_EXPR, TREE_TYPE (x),
+                         ubsan_instrument_vla (input_location, x), x);
+             finish_expr_stmt (x);

Saving 'size' here doesn't help since it's already been used above. 
Could you use itype instead of size here?

Jason