[cfe-commits] [PATCH] Add android triple to GCC Installation Detector

[Logan Chien]

Thanks for your comments.
Let me fix the "-B" priority problem first.
I will refine this patch after I have fixed "-B" problem.

Logan

On Thu, Sep 13, 2012 at 12:02 AM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:

> On 12 September 2012 01:33, Logan Chien <tzuhsiang.chien at gmail.com> wrote:
> > Dear all,
> >
> >   There's the rebased new patch to follow the test input change.
> > Every code remained unchanged.  Please have a look.  Thanks.
>
>
> Why do you need
> test/Driver/Inputs/basic_android_tree/i686-linux-android/bin/ar?
>
> +// FIXME: If you fail this regression test, please check whether the GCC
> in
> +// "/usr" is newer than 4.4.3 or 4.6 or not.  This unit test not good
> enough
> +// because clang will try to search for latest "as" or "ld" at "/usr" even
> +// though "-B" is specified.
>
>
> This is likely to fail in the bots. You will probably need to use a
> less strict regular expression until this is fixed.
>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120913/34fcc015/attachment.html>