Re: ASAN merge...

[Richard Earnshaw <rearnsha at arm dot com>]

On 13/11/12 22:41, Andrew Pinski wrote:
> On Tue, Nov 13, 2012 at 2:31 PM, Ramana Radhakrishnan
> <ramana.gcc@googlemail.com> wrote:
> > On 13 Nov 2012, at 21:18, Konstantin Serebryany <konstantin.s.serebryany@gmail.com> wrote:
> >
> > > On Tue, Nov 13, 2012 at 8:21 AM, Diego Novillo <dnovillo@google.com> wrote:
> > > > On Tue, Nov 13, 2012 at 12:07 AM, David Miller <davem@davemloft.net> 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 should just be
> > >
> > > 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 ?
>
> And this arm is really arm32 and not ARM64 :).  People need to start
> calling it arm32 to make sure they don't get confused between the two
> different architectures.

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 AArch64 aren't that much harder to type and 
they are completely unambiguous.

If nothing else, remember that there were arm3 and arm6 chips at one 
time, so we have patterns that match arm3* and arm6* to match those CPUs 
(because people keep insisting on adding miscellaneous junk at the end 
of the CPU field).  At all costs we need to avoid getting false matches 
in here, or chaos will ensue.

The architecture overall is ARM, the operating states within it are 
AArch32 and AArch64.  There is no ARM32 and there is no ARM64.

R.

> Thanks
> Andrew
>
>
> > > But I'd suggest not to worry about any platforms other than
> > > x86_{32,64}/Linux until the whole thing
> > > (including tests) is fully integrated.
> >
> >
> > Ramana
> >
> > > --kcc
> > >
> > > > completely disabled until the other ports start showing up.
> > > >
> > > > Dodji is your patch committed?
> > > >
> > > >
> > > > Diego.