[cfe-commits] Support Android standalone toolchain in Clang driver

Evgeniy Stepanov eugeni.stepanov at gmail.com
Tue Aug 28 04:07:19 PDT 2012


Sorry, this is, finally, the right version.

On Tue, Aug 28, 2012 at 2:53 PM, Evgeniy Stepanov
<eugeni.stepanov at gmail.com> wrote:
> On Tue, Aug 28, 2012 at 1:56 PM, Evgeniy Stepanov
> <eugeni.stepanov at gmail.com> wrote:
>> Rafael, thanks for your consideration!
>>
>> I forgot to include the test for the C++ include dir in the latest
>> patch. Attaching a full patch this time.
>>
>> Believe it or not, but Inputs/basic_android_tree/sysroot does exist.
>> SVN is just not smart enough to show it in the patch. Here is what I
>> plan to commit, if approved:
>>
>> M       test/Driver/asan-ld.c
>> A       test/Driver/android-standalone.cpp
>> A       test/Driver/Inputs/basic_android_tree/sysroot
>> A  +    test/Driver/Inputs/basic_android_tree/sysroot/usr
>> A       test/Driver/Inputs/basic_android_tree/lib
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi/4.4.3
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi/4.4.3/crtendS.o
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi/4.4.3/crtbegin.o
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi/4.4.3/crtend.o
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi/4.4.3/crtbeginS.o
>> A       test/Driver/Inputs/basic_android_tree/lib/gcc/arm-linux-androideabi/4.4.3/crtbeginT.o
>> D       test/Driver/Inputs/basic_android_tree/usr
>> D       test/Driver/Inputs/basic_android_tree/usr/lib
>> D       test/Driver/Inputs/basic_android_tree/usr/lib/crtbegin_static.o
>> D       test/Driver/Inputs/basic_android_tree/usr/lib/crtbegin_so.o
>> D       test/Driver/Inputs/basic_android_tree/usr/lib/crtend_android.o
>> D       test/Driver/Inputs/basic_android_tree/usr/lib/crtend_so.o
>> D       test/Driver/Inputs/basic_android_tree/usr/lib/crtbegin_dynamic.o
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/include
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/include/c++
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/include/c++/4.4.3
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/include/c++/4.4.3/.keep
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/lib
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/lib/.keep
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/bin
>> A       test/Driver/Inputs/basic_android_tree/arm-linux-androideabi/bin/.keep
>> M       test/Driver/linux-ld.c
>> M       lib/Driver/ToolChains.cpp
>>
>> All regular files under basic_android_tree are empty.
>>
>> On Mon, Aug 27, 2012 at 8:19 PM, Rafael EspĂ­ndola
>> <rafael.espindola at gmail.com> wrote:
>>> On 27 August 2012 07:50, Evgeniy Stepanov <eugeni.stepanov at gmail.com> wrote:
>>>> I wonder if someone could take a look at this trivial patch? It is
>>>> absolutely required to build any kind of C++ code with the standalone
>>>> toolchain from Android NDK.
>>>
>>> Can you add a test showing the new C++ include location being used?
>>>
>>> The directory Inputs/basic_android_tree/sysroot doesn't exist, right?
>>> Is that a intended feature? If not, could you please create it?
>>>
>>> Cheers,
>>> Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: android-standalone.patch
Type: application/octet-stream
Size: 7127 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120828/1458db76/attachment.obj>


More information about the cfe-commits mailing list