On Tue, Nov 13, 2012 at 2:40 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Tue, Nov 13, 2012 at 2:39 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
>> On Tue, Nov 13, 2012 at 2:26 PM, Konstantin Serebryany
>> <konstantin.s.serebry...@gmail.com> wrote:
>>> H.J.,
>>> Question about this patch.
>>> Will it work if we simply replace
>>>    #if __WORDSIZE == 64
>>> with
>>>   #ifdef x86_64
>>> ?
>>>
>>> Today, x86_64 is the only 64-bit architecture supported by asan
>>> run-time on linux anyway.
>>>
>>>
>>
>> That works for me.
>>
>
> It should be
>
> #ifdef __x86_64__

Sure. Done, LLVM r167883.


>
> not
>
> #ifdef x86_64
>
> --
> H.J.

Reply via email to