Yes, but still seems to add API and code around something that isn't worth promoting:<div>--</div><div><div>diffstat t.patch</div><div> ToolChains.cpp |  100 +++++++++++++++++++++++++++++++++++----------------------</div>
<div> Tools.cpp      |   97 +++++++++++++++++++++++++++++++++++++++++++++----------</div><div> Tools.h        |   34 +++++++++++++++++++</div><div> 3 files changed, 177 insertions(+), 54 deletions(-)</div></div><div>--</div>
<div><br></div><div> - Daniel</div><div><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 31, 2012 at 3:51 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.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="im">On 31 October 2012 13:55, Daniel Dunbar <<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>> wrote:<br>

> I think very little of the char* handling in the driver is related to this<br>
> particular issue.<br>
><br>
> I'm hesitant to promote the very specialized Darwin CPU arch naming scheme<br>
> higher in the API. This isn't really a useful scheme (i.e., no one else<br>
> should use this notion), but is required for Darwin compatibility.<br>
><br>
> If anything, I'd rather go the other direction and localize<br>
> getArchTypeForDarwinArchName into Clang.<br>
<br>
</div>Those are fairly independent changes. I have moved<br>
getArchTypeForDarwinArchName to lib/Driver. The attached patch is<br>
rebased on top of that. Note that only this patch already removes a<br>
"switch" over strings (converts one StringSwitch and a if chain into<br>
one StringSwitch.).<br>
<br>
>  - Daniel<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>