[llvm-commits] PATCH: Build both 32-bit and 64-bit ASan runtime on x86_64 (in configure+make build)

[Daniel Dunbar]

Hi Alexey,

The problem with this is that not all Linux platforms are going to have
multi arch support installed. To get this to work you will need to also add
some configure checks to make sure that that can be used, I think?

FWIW, I tend to ascribe complete ownership for what happens on Linux to you
guys, so I don't necessarily think you need my review on such patches. I'm
not generally up to speed with what the state of the Linux world is.

 - Daniel

On Monday, September 24, 2012, Alexander Potapenko wrote:

> +Nico FHI
>
> On Fri, Sep 21, 2012 at 8:26 PM, Alexey Samsonov <samsonov at google.com<javascript:;>>
> wrote:
> > Hi Daniel,
> >
> > These two patches allow building i386 ASan runtime if target arch is
> x86_64.
> > Typically we want both runtimes to reside
> > in lib/clang/3.2/lib/linux/ so that "clang++ -faddress-sanitizer" could
> work
> > with both -m64 and -m32. Is it ok to commit?
> >
> > --
> > Alexey Samsonov, MSK
> >
>
>
>
> --
> Alexander Potapenko
> Software Engineer
> Google Moscow
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120925/99481f1b/attachment.html>