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

Evgeniy Stepanov eugeni.stepanov at gmail.com
Tue Aug 28 03:53:15 PDT 2012


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: 7133 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120828/08d0f157/attachment.obj>


More information about the cfe-commits mailing list