[llvm-commits] [compiler-rt] r163313 - /compiler-rt/trunk/lib/asan/tests/asan_test.cc

Evgeniy Stepanov eugeni.stepanov at gmail.com
Fri Sep 14 02:02:52 PDT 2012


On Fri, Sep 14, 2012 at 5:16 AM, Eric Christopher <echristo at apple.com> wrote:
>
> On Sep 6, 2012, at 5:50 AM, Evgeniy Stepanov <eugeni.stepanov at gmail.com> wrote:
>
>> -#if !defined(__APPLE__) && !defined(ANDROID)
>> +#if !defined(__APPLE__) && !defined(ANDROID) && !defined(__ANDROID__)
>
> You can have either of these defined and not both?

We can have both when this code is being built as part of Android
platform tree. Anyway, either one of them means we are on Android and
don't have posix_memalign.

ANDROID without underscores is legacy. Recent gcc and clang both have
__ANDROID__.

>
>>   // fprintf(stderr, "posix_memalign\n");
>
> Debugging code?

r163881

>
> -eric



More information about the llvm-commits mailing list