Re: ASAN merge...

2012-11-17 Thread Toon Moene
On 11/16/2012 04:35 PM, Ian Lance Taylor wrote: I expect it's pronounced with a sort of throat-clearing noise that is hard to write. Sort of like the gargling sound represented by "argh." argh32 and argh64. I thought it was a nice way to enshrine a Monty Python joke into silicon. -- Toon M

Re: ASAN merge...

2012-11-16 Thread Richard Earnshaw
On 16/11/12 15:35, Ian Lance Taylor wrote: On Fri, Nov 16, 2012 at 1:54 AM, Andrew Haley wrote: On 11/14/2012 01:49 PM, Richard Earnshaw wrote: Please please don't get into the habit of calling it ARM32 and ARM64, you're just sowing confusion; there are good reasons why those names weren't ad

Re: ASAN merge...

2012-11-16 Thread Ian Lance Taylor
On Fri, Nov 16, 2012 at 1:54 AM, Andrew Haley wrote: > On 11/14/2012 01:49 PM, Richard Earnshaw wrote: > >> Please please don't get into the habit of calling it ARM32 and ARM64, >> you're just sowing confusion; there are good reasons why those names >> weren't adopted (some technical, some not) an

Re: ASAN merge...

2012-11-16 Thread Andrew Haley
On 11/14/2012 01:49 PM, Richard Earnshaw wrote: > Please please don't get into the habit of calling it ARM32 and ARM64, > you're just sowing confusion; there are good reasons why those names > weren't adopted (some technical, some not) and I'm not about to rehash > them all now. AArch32 and AA

Re: ASAN merge...

2012-11-14 Thread David Miller
From: Dodji Seketeli Date: Wed, 14 Nov 2012 14:26:40 +0100 > I guess we could do that. That would build libsanitizer, but asan will > still not be available on sparc if the asan_shadow_offset() target hook > is not provided. Is that OK to you? Yes.

Re: ASAN merge...

2012-11-14 Thread Richard Earnshaw
On 13/11/12 22:41, Andrew Pinski wrote: On Tue, Nov 13, 2012 at 2:31 PM, Ramana Radhakrishnan wrote: On 13 Nov 2012, at 21:18, Konstantin Serebryany wrote: On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo wrote: On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: This has broken the

Re: ASAN merge...

2012-11-14 Thread Dodji Seketeli
David Miller writes: > From: Diego Novillo > Date: Tue, 13 Nov 2012 11:21:59 -0500 > >> On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: >>> >>> This has broken the build on every Linux target that hasn't added >>> the necessary cpu specific code to asan_linux.cc >> >> This should be fixe

Re: ASAN merge...

2012-11-14 Thread Dodji Seketeli
Diego Novillo writes: > On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: >> >> This has broken the build on every Linux target that hasn't added >> the necessary cpu specific code to asan_linux.cc > > This should be fixed by Dodji's recent patch. ASAN is not currently > ported to any targe

Re: ASAN merge...

2012-11-13 Thread Ramana Radhakrishnan
On Tue, Nov 13, 2012 at 10:41 PM, Andrew Pinski wrote: > On Tue, Nov 13, 2012 at 2:31 PM, Ramana Radhakrishnan > wrote: >> >> >> On 13 Nov 2012, at 21:18, Konstantin Serebryany >> wrote: >> >>> On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo wrote: On Tue, Nov 13, 2012 at 12:07 AM, David M

Re: ASAN merge...

2012-11-13 Thread Konstantin Serebryany
+euge...@google.com >>> >>> asan run-time (and the LLVM part) works on Mac and on ARM/Linux. >> >> And when you say ARM / Linux, has this been tested on older versions of the >> architecture or just v7-a ? Evgeniy is better equipped to answer this. At some point we've been testing asan on the ac

Re: ASAN merge...

2012-11-13 Thread Andrew Pinski
On Tue, Nov 13, 2012 at 2:31 PM, Ramana Radhakrishnan wrote: > > > On 13 Nov 2012, at 21:18, Konstantin Serebryany > wrote: > >> On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo wrote: >>> On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: This has broken the build on every Linux ta

Re: ASAN merge...

2012-11-13 Thread Ramana Radhakrishnan
On 13 Nov 2012, at 21:18, Konstantin Serebryany wrote: > On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo wrote: >> On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: >>> >>> This has broken the build on every Linux target that hasn't added >>> the necessary cpu specific code to asan_linux

Re: ASAN merge...

2012-11-13 Thread Konstantin Serebryany
On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo wrote: > On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: >> >> This has broken the build on every Linux target that hasn't added >> the necessary cpu specific code to asan_linux.cc > > This should be fixed by Dodji's recent patch. ASAN is not

Re: ASAN merge...

2012-11-13 Thread David Miller
From: Diego Novillo Date: Tue, 13 Nov 2012 11:21:59 -0500 > On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: >> >> This has broken the build on every Linux target that hasn't added >> the necessary cpu specific code to asan_linux.cc > > This should be fixed by Dodji's recent patch. ASAN i

Re: ASAN merge...

2012-11-13 Thread H.J. Lu
On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo wrote: > On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: >> >> This has broken the build on every Linux target that hasn't added >> the necessary cpu specific code to asan_linux.cc > > This should be fixed by Dodji's recent patch. ASAN is not

Re: ASAN merge...

2012-11-13 Thread Diego Novillo
On Tue, Nov 13, 2012 at 12:07 AM, David Miller wrote: > > This has broken the build on every Linux target that hasn't added > the necessary cpu specific code to asan_linux.cc This should be fixed by Dodji's recent patch. ASAN is not currently ported to any target other than x86/linux, so it shou