On Sat, Sep 20, 2014 at 10:42:01PM -0400, Jason Merrill wrote:
> On 09/18/2014 10:12 AM, Jakub Jelinek wrote:
> >Clobber stmts, being artificial statements, were certainly never
> >meant to be instrumented. In 4.8 when asan has been introduced into gcc,
> >the lhs of clobber could be only a decl and as a whole decl store would not
> >be really instrumented, but with *this clobbers in 4.9 that is no longer the
> >case.
>
> And this patch fixes the clobber to only clobber the as-base part of the
> class, since any virtual bases might not be destroyed yet. This patch also
> fixes Jakub's testcase (without his patch).
Thanks.
> commit e8c36b9fe6c1534925d809a7a15e599770466ee3
> Author: Jason Merrill <ja...@redhat.com>
> Date: Thu Sep 18 11:15:57 2014 -0400
>
> PR c++/62017
> * decl.c (begin_destructor_body): Only clobber the as-base part of
> *this.
Jakub
