LGTM, thanks for the explanation and the comments!<div><br></div><div>Also, yay test cases!<br><div><br></div><div>One grammar nit:</div></div><div><br></div><div><div>+  // targets. It contains N32 ABI binaries. Do not</div>
<div>+  // use this folder if produce code for other ABI.</div></div><div><br></div><div>"Do not use this folder when producing code for the normal ABI."? However you want to phrase "normal ABI", I'm assuming you mean anything other than N32 ABI.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 13, 2012 at 1:20 PM, 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">Hi,<br>
<br>
Clang driver considers lib32 directory as a usual folder with 32-bit<br>
binaries. That is not correct for MIPS where this directory contains<br>
N32 ABI binaries. N32 is a minor variation on the high performance<br>
64-bit ABI and is not used to compile 32-bit code. That's why Clang<br>
passes incorrect paths to the linker if user has MIPS multilib<br>
environment and wants to compile 32-bit code.<br>
<br>
The attached patch fixes this problem. It does not use "lib32"<br>
directory to create a library/object files paths list when target is<br>
MIPS 32-bit. Later I plan to implement full N32 ABI support and<br>
provide more test cases.<br>
<br>
Please review the patch.<br>
<br>
--<br>
Simon<br>
<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>
<br></blockquote></div><br></div>