On Wed, Nov 27, 2013 at 11:29:21AM +0100, Marek Polacek wrote:
> This was meant only as a TLC, but it also fixes PR59306. Using
> walk_gimple_op was an overkill; gimple_{store,assign_load}_p is
> enough. As a side effect, it also fixes the bug because now we
> better restrict what goes into instrument_member_call.
>
> Bootstrapped, ran ubsan testsuite on x86_64-linux, ok for trunk?
Ok, thanks.
> 2013-11-27 Marek Polacek <pola...@redhat.com>
>
> PR sanitizer/59306
> * ubsan.c (instrument_null): Use gimple_store_p/gimple_assign_load_p
> instead of walk_gimple_op.
> (ubsan_pass): Adjust. Call instrument_null only if SANITIZE_NULL.
> testsuite/
> * g++.dg/ubsan/pr59306.C:
Missing " New test." ;)
Jakub
