Thanks.  LGTM.  May you commit this for me?  Thanks.<br><br>- Logan<br><br><br><div class="gmail_quote">On Wed, Aug 29, 2012 at 9:58 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>
<div><br>
On Wed, Aug 29, 2012 at 2:35 PM, Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com" target="_blank">tzuhsiang.chien@gmail.com</a>> wrote:<br>
>   It seems that clang is not pre-defining _MIPS_ARCH as gcc did.<br>
> This may cause some problem when we are using clang as the<br>
> cross compiler.  The attached patch contains a simple fix and the<br>
> test cases.  Please have a look.  Thanks.<br>
<br>
</div>LGTM<br>
<br>
Just a couple of notes:<br>
1. I suggest to replace wildcards by real processor names in the test.<br>
If/when we change default processor names, we will change the test<br>
accordingly.<br>
2. CPU variable is std::string so it is not necessary to use explicit<br>
cast in the following expression:<br>
std::string("\"") + CPU + "\""<br>
<br>
Any objections?<br>
<br>
--<br>
Simon<br>
</blockquote></div><br>