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

[Daniel Dunbar]

On Thu, Sep 27, 2012 at 8:08 AM, Alexey Samsonov <samsonov at google.com> wrote:
> 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.

I'm fine with it, for the record.

 - Daniel

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