[cfe-commits] [PATCH] Fix x86 android support in clang

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Oct 31 12:34:24 PDT 2012


On 31 October 2012 15:12, Vane, Edwin <edwin.vane at intel.com> wrote:
>> -        CPUName = "pentium4";
>> +        CPUName = isAndroid ? "atom" : "pentium4";
>
> This matches gcc behavior? Can you add a test for this part?
>
> REV: Sure but what test case would I adjust? These options only seem to show up for compiling and I couldn't find a test suite for testing android compilation. I'll double-check with gcc.

You can test the -target-cpu that the driver is passing to "clang -cc1", no?

Cheers,
Rafael



More information about the cfe-commits mailing list