Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-13 Thread Willy Tarreau
On Sun, Jun 14, 2020 at 01:20:43AM +0500, ??? wrote: > I added "-O1" to travis builds. > Can we apply it until a better solution will be found ? Applied, thank you Ilya. Willy

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-13 Thread Илья Шипицин
I added "-O1" to travis builds. Can we apply it until a better solution will be found ? пт, 12 июн. 2020 г. в 21:40, Илья Шипицин : > > > пт, 12 июн. 2020 г. в 21:09, Willy Tarreau : > >> On Fri, Jun 12, 2020 at 08:57:44PM +0500, ??? wrote: >> > ??, 12 ???. 2020 ?. ? 20:46, Willy

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Илья Шипицин
пт, 12 июн. 2020 г. в 21:09, Willy Tarreau : > On Fri, Jun 12, 2020 at 08:57:44PM +0500, ??? wrote: > > ??, 12 ???. 2020 ?. ? 20:46, Willy Tarreau : > > > > > On Fri, Jun 12, 2020 at 08:11:52PM +0500, ??? wrote: > > > > > Has it ever reported a *real* issue ? I mean, we've been

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Willy Tarreau
On Fri, Jun 12, 2020 at 08:57:44PM +0500, ??? wrote: > ??, 12 ???. 2020 ?. ? 20:46, Willy Tarreau : > > > On Fri, Jun 12, 2020 at 08:11:52PM +0500, ??? wrote: > > > > Has it ever reported a *real* issue ? I mean, we've been working around > > > > > > > > > > > > >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Илья Шипицин
пт, 12 июн. 2020 г. в 20:46, Willy Tarreau : > On Fri, Jun 12, 2020 at 08:11:52PM +0500, ??? wrote: > > > Has it ever reported a *real* issue ? I mean, we've been working around > > > > > > > > > https://github.com/haproxy/haproxy/issues/96 > >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Willy Tarreau
On Fri, Jun 12, 2020 at 08:11:52PM +0500, ??? wrote: > > Has it ever reported a *real* issue ? I mean, we've been working around > > > > > https://github.com/haproxy/haproxy/issues/96 > https://github.com/haproxy/haproxy/issues/104 > https://github.com/haproxy/haproxy/issues/106 >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Илья Шипицин
пт, 12 июн. 2020 г. в 20:01, Willy Tarreau : > On Fri, Jun 12, 2020 at 07:52:48PM +0500, ??? wrote: > > it should be detectable using > > > > #if defined(__has_feature)# if __has_feature(address_sanitizer)// > > code that builds only under AddressSanitizer# endif#endif > > OK that

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Willy Tarreau
On Fri, Jun 12, 2020 at 07:52:48PM +0500, ??? wrote: > it should be detectable using > > #if defined(__has_feature)# if __has_feature(address_sanitizer)// > code that builds only under AddressSanitizer# endif#endif OK that could be useful indeed, thanks! > I agree to remove asan from

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Илья Шипицин
пт, 12 июн. 2020 г. в 19:31, Willy Tarreau : > Hi Ilya, > > On Mon, Mar 16, 2020 at 10:49:26AM +0100, Tim Düsterhus wrote: > > Ilya, > > > > Am 16.03.20 um 07:52 schrieb ???: > > > we use clang because of its address sanitizer. I found gcc asan more > noisy > > > and less usable. > >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-06-12 Thread Willy Tarreau
Hi Ilya, On Mon, Mar 16, 2020 at 10:49:26AM +0100, Tim Düsterhus wrote: > Ilya, > > Am 16.03.20 um 07:52 schrieb ???: > > we use clang because of its address sanitizer. I found gcc asan more noisy > > and less usable. Going back to this, I spent the whole day trying to figure what

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Илья Шипицин
oops :) Comment #4 on issue 323 by david...@chromium.org: boringssl fails on travis-ci/clang-9 https://bugs.chromium.org/p/boringssl/issues/detail?id=323#c4 Looks like the issue is you've managed to convince it that your C compiler is Clang and your C++ compiler is GCC. -- The C compiler

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Илья Шипицин
пн, 16 мар. 2020 г. в 14:54, Willy Tarreau : > On Mon, Mar 16, 2020 at 10:49:26AM +0100, Tim Düsterhus wrote: > > Ilya, > > > > Am 16.03.20 um 07:52 schrieb ???: > > > we use clang because of its address sanitizer. I found gcc asan more > noisy > > > and less usable. > > > > I believe

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Willy Tarreau
On Mon, Mar 16, 2020 at 10:49:26AM +0100, Tim Düsterhus wrote: > Ilya, > > Am 16.03.20 um 07:52 schrieb ???: > > we use clang because of its address sanitizer. I found gcc asan more noisy > > and less usable. > > I believe you broke ASAN with clang-9 anyway, because the Travis >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Tim Düsterhus
Ilya, Am 16.03.20 um 07:52 schrieb Илья Шипицин: > we use clang because of its address sanitizer. I found gcc asan more noisy > and less usable. I believe you broke ASAN with clang-9 anyway, because the Travis configuration checks for 'clang':

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Илья Шипицин
пн, 16 мар. 2020 г. в 13:40, Willy Tarreau : > On Mon, Mar 16, 2020 at 12:51:20PM +0500, ??? wrote: > > From 5d5891166389dc03ab4fb63ca9edaa35feca8fcc Mon Sep 17 00:00:00 2001 > > From: Ilya Shipitsin > > Date: Mon, 16 Mar 2020 12:49:34 +0500 > > Subject: [PATCH] CI: switch BoringSSL

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Willy Tarreau
On Mon, Mar 16, 2020 at 12:51:20PM +0500, ??? wrote: > From 5d5891166389dc03ab4fb63ca9edaa35feca8fcc Mon Sep 17 00:00:00 2001 > From: Ilya Shipitsin > Date: Mon, 16 Mar 2020 12:49:34 +0500 > Subject: [PATCH] CI: switch BoringSSL back to clang-7 > > it turned out that BoringSSL is not

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Илья Шипицин
пн, 16 мар. 2020 г. в 12:09, Willy Tarreau : > On Mon, Mar 16, 2020 at 11:52:27AM +0500, ??? wrote: > > ??, 16 ???. 2020 ?. ? 11:35, Willy Tarreau : > > > > > On Sun, Mar 15, 2020 at 10:54:56PM +0500, ??? wrote: > > > > ??, 14 ???. 2020 ?. ? 14:23, Willy Tarreau : > > > > > > >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Willy Tarreau
On Mon, Mar 16, 2020 at 11:52:27AM +0500, ??? wrote: > ??, 16 ???. 2020 ?. ? 11:35, Willy Tarreau : > > > On Sun, Mar 15, 2020 at 10:54:56PM +0500, ??? wrote: > > > ??, 14 ???. 2020 ?. ? 14:23, Willy Tarreau : > > > > > > > Hi Ilya, > > > > > > > > On Fri, Jan 24, 2020 at

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Илья Шипицин
пн, 16 мар. 2020 г. в 11:35, Willy Tarreau : > On Sun, Mar 15, 2020 at 10:54:56PM +0500, ??? wrote: > > ??, 14 ???. 2020 ?. ? 14:23, Willy Tarreau : > > > > > Hi Ilya, > > > > > > On Fri, Jan 24, 2020 at 11:46:45AM +0500, ??? wrote: > > > > Hello, > > > > > > > > let us use

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-16 Thread Willy Tarreau
On Sun, Mar 15, 2020 at 10:54:56PM +0500, ??? wrote: > ??, 14 ???. 2020 ?. ? 14:23, Willy Tarreau : > > > Hi Ilya, > > > > On Fri, Jan 24, 2020 at 11:46:45AM +0500, ??? wrote: > > > Hello, > > > > > > let us use clang-9 instead of default clang-7 for linux builds. > > > > It

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-15 Thread Илья Шипицин
сб, 14 мар. 2020 г. в 14:23, Willy Tarreau : > Hi Ilya, > > On Fri, Jan 24, 2020 at 11:46:45AM +0500, ??? wrote: > > Hello, > > > > let us use clang-9 instead of default clang-7 for linux builds. > > It seems I missed this one. Now applied carefully, we'll see. If it > causes new

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-15 Thread Martin Grigorov
Hi, On Sat, Mar 14, 2020 at 11:26 AM Willy Tarreau wrote: > Hi Ilya, > > On Fri, Jan 24, 2020 at 11:46:45AM +0500, ??? wrote: > > Hello, > > > > let us use clang-9 instead of default clang-7 for linux builds. > > It seems I missed this one. Now applied carefully, we'll see. If it >

Re: [PATCH] switch to clang-9 in Linux/travis-ci builds

2020-03-14 Thread Willy Tarreau
Hi Ilya, On Fri, Jan 24, 2020 at 11:46:45AM +0500, ??? wrote: > Hello, > > let us use clang-9 instead of default clang-7 for linux builds. It seems I missed this one. Now applied carefully, we'll see. If it causes new failures, we'll adjust accordingly. Willy