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

[Alexey Samsonov]

On Tue, Sep 25, 2012 at 10:56 PM, Daniel Dunbar <daniel at zuster.org> wrote:

> 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.
>

+eugenis

BTW, eugenis is also eager to leverage compiler-rt build system for
Android, so if you're really ok with granting us permission to commit
linux-specific changes, we can speed up a bit.


>
>  - 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>
>> 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
>>
>


-- 
Alexey Samsonov, MSK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120927/e8a3f9cc/attachment.html>