LGTM.<br><br>But I would suggest that re-use the CHECK-ANDROID, CHECK-ANDROID-SO,<br>CHECK-ANDROID-STATIC, and CHECK-ANDROID_PIE in linux-ld.c (Check the<br>attached patch for details)<br><br>Sincerely,<br>Logan<br><br><div class="gmail_quote">
On Tue, Oct 2, 2012 at 6:00 AM, Simon Atanasyan <span dir="ltr"><<a href="mailto:satanasyan@mips.com" target="_blank">satanasyan@mips.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Sun, Sep 30, 2012 at 11:42 PM, Simon Atanasyan <<a href="mailto:satanasyan@mips.com">satanasyan@mips.com</a>> wrote:<br>
><br>
> On Sat, Sep 29, 2012 at 12:04 AM, Simon Atanasyan <<a href="mailto:satanasyan@mips.com">satanasyan@mips.com</a>> wrote:<br>
>> The patch adds "mipsel-linux-android" to the list of valid MIPS target<br>
>> triples. That simplifies usage of clang with Android NDK.<br>
><br>
> Here is the second version of the patch. Now Clang driver constructs a<br>
> correct path to crt*.o object files if target is mipsel-linux-android<br>
> and arcitecture/CPU is MIPSr2.<br>
><br>
> Please review the patch.<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">Ping.<br>
<br>
--<br>
Simon<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br>