Thanks for your comments. Here's the new patch. The test cases is included now.
However, IMHO, this new test case is not very reliable. The GCCInstallationDetector will search for system root (i.e. /usr) for latest (newest) GCC, thus "-B" option may not be used if we have older GCC version in the test case. Maybe we have to revise the "-B" implementation as well. Any further suggestion? Thanks. Sincerely, Logan On Wed, Aug 29, 2012 at 3:23 PM, Chandler Carruth <[email protected]>wrote: > If you want to teach the Clang driver about the android toolchains, you'll > need to add some test cases for them. You can find the test case strategy > by looking at the tests for Linux and FreeBSD; I think they're pretty > comprehensive at this point. > > > On Wed, Aug 29, 2012 at 12:19 AM, Logan Chien > <[email protected]>wrote: > >> Hi, >> >> Currently, the GCCInstallationDetector is not looking for >> mipsel-linux-android and >> i686-linux-android. As the result, the clang driver can't use Android >> Mips/X86 toolchain >> even if -B is specified. I hope this patch can solve this issue. Please >> review. Thanks. >> >> Sincerely, >> Logan >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> >> >
Add-Android-triple-for-X86-Mips-GCC-detection.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
