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

Evgeniy Stepanov eugeni.stepanov at gmail.com
Tue Aug 28 02:56:06 PDT 2012


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


More information about the cfe-commits mailing list