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
<[email protected]> wrote:
> On 27 August 2012 07:50, Evgeniy Stepanov <[email protected]> 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

Attachment: android-standalone.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to