Re: [PATCH 2/7] Libsanitizer merge from upstream r249633.

2015-10-19 Thread Jakub Jelinek
On Thu, Oct 15, 2015 at 01:34:06PM +0300, Maxim Ostapenko wrote: > Ah, right, fixing this now. Does this looks better now? Yes, it is ok now. > 2015-10-12 Maxim Ostapenko > > config/ > > * bootstrap-asan.mk: Replace ASAN_OPTIONS=detect_leaks with >

Re: [PATCH 2/7] Libsanitizer merge from upstream r249633.

2015-10-15 Thread Maxim Ostapenko
On 14/10/15 10:30, Jakub Jelinek wrote: On Tue, Oct 13, 2015 at 02:16:23PM +0300, Maxim Ostapenko wrote: This patch introduces required compiler changes. Now, we don't version asan_init, we have a special __asan_version_mismatch_check_v[n] symbol for this. For this, I just have to wonder what

Re: [PATCH 2/7] Libsanitizer merge from upstream r249633.

2015-10-14 Thread Jakub Jelinek
On Tue, Oct 13, 2015 at 02:16:23PM +0300, Maxim Ostapenko wrote: > This patch introduces required compiler changes. Now, we don't version > asan_init, we have a special __asan_version_mismatch_check_v[n] symbol for > this. For this, I just have to wonder what is the actual improvement over what

Re: [PATCH 2/7] Libsanitizer merge from upstream r249633.

2015-10-14 Thread Yury Gribov
On 10/13/2015 02:16 PM, Maxim Ostapenko wrote: This patch introduces required compiler changes. Now, we don't version asan_init, we have a special __asan_version_mismatch_check_v[n] symbol for this. Also, asan_stack_malloc_[n] doesn't take a local stack as a second parameter anymore, so don't

[PATCH 2/7] Libsanitizer merge from upstream r249633.

2015-10-13 Thread Maxim Ostapenko
This patch introduces required compiler changes. Now, we don't version asan_init, we have a special __asan_version_mismatch_check_v[n] symbol for this. Also, asan_stack_malloc_[n] doesn't take a local stack as a second parameter anymore, so don't pass it. 2015-10-12 Maxim Ostapenko