Hi Anton,<br><br>  If you have some concern with the triple parser change, then I'm<br>sure that clang or llc still supports arm-linux-androideabi, the test<br>cases with androideabi has been remained unchanged.  And we<br>
are still passing every llvm/clang regression test.  Please let me know<br>if you have other concerns?  Thanks.<br><br>Sincerely,<br>- Logan<br><br><div class="gmail_quote">On Fri, Aug 31, 2012 at 4:32 PM, Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">>   To avoid the confusion of Android environment on MIPS/X86 architecture, I<br>
> believe that<br>
> it is better to replace llvm::Triple::ANDROIDEABI with<br>
> llvm::Triple::Android.  So that we<br>
> can use mipsel-linux-android instead of mipsel-linux-androideabi (this<br>
> applies to x86 as well).<br>
> Please review.  Thanks for your help.<br>
</div>This seems to break backward compatibility. You need to handle<br>
androideabi at least on ARM.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
With best regards, Anton Korobeynikov<br>
Faculty of Mathematics and Mechanics, Saint Petersburg State University<br>
</font></span></blockquote></div><br>