[PATCH] D18167: [ELF/Driver] Avoid a reference to the ELFOptTable after it has gone out of scope

[Kostya Serebryany via llvm-commits]

Cool!
please make sure that the LLVM test harness does not unset ASAN_OPTIONS

On Tue, Mar 15, 2016 at 4:17 PM, Filipe Cabecinhas <
filcab+llvm.phabricator at gmail.com> wrote:

> Hi Kostya,
>
> ASan-ized lld gets past check-lld without problems, so we might not
> need any cleanup :-)
>
> Thank you,
>
>   Filipe
>
> On Tue, Mar 15, 2016 at 3:52 PM, Kostya Serebryany <kcc at google.com> wrote:
> >
> >
> > On Mon, Mar 14, 2016 at 7:29 PM, Filipe Cabecinhas
> > <filcab+llvm.phabricator at gmail.com> wrote:
> >>
> >> It looks like it should, but I can't try it out now.
> >> I can try it tonight or tomorrow morning at most. If you want, you can
> >> compile llvm+lld with ASan and turn on the Asan option (double-check
> that
> >> you catch the bug first ;-) ).
> >>
> >> CCing Kostya:
> >> Should we add ASAN_OPTIONS=detect_stack_use_after_free=true to the
> >> sanitizer bot?
> >
> >
> > That would be lovely!
> > Any volunteers for the cleanup?
> >
> >>
> >> It would have caught this bug a long time ago.
> >> We might also need to do some cleanup before we can do it.
> >>
> >> Thank you,
> >>
> >>   Filipe
> >>
> >>
> >>
> >> On Monday, 14 March 2016, Rui Ueyama <ruiu at google.com> wrote:
> >>>
> >>> ruiu added a comment.
> >>>
> >>> Does http://reviews.llvm.org/D18169 solves PR26908?
> >>>
> >>>
> >>> http://reviews.llvm.org/D18167
> >>>
> >>>
> >>>
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160315/3b5f949c/attachment.html>