Re: [PATCH] Fix up bogus warning (PR sanitizer/59331)

[Jason Merrill]

On 11/28/2013 12:14 PM, Marek Polacek wrote:

              t = fold_build2 (COMPOUND_EXPR, TREE_TYPE (t),
-                              ubsan_instrument_vla (input_location, t), t);
+                              ubsan_instrument_vla (input_location, t),
+                              /* Cast to void to prevent bogus warning.  */
+                              build1 (CONVERT_EXPR, void_type_node, t));
              finish_expr_stmt (t);

Why do you need the COMPOUND_EXPR at all?  Why can't you just do

t = ubsan_instrument_vla (input_location, t);

?

Jason