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

2013-11-29 Thread Marek Polacek
On Thu, Nov 28, 2013 at 07:04:43PM -0500, Jason Merrill wrote: 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

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

2013-11-29 Thread Jason Merrill
OK. Jason

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

2013-11-28 Thread Marek Polacek
We wrongly warned on instrumented VLAs that the size expression's value is not used (with cc1plus only). Unfortunately, this hasn't been detected before due to disabled warnings in the VLA tests. This patch adds a (void) cast to suppress the warning as well as enables the warnings in the VLA

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

2013-11-28 Thread 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