<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Good idea! Done in svn r141344.<div><br><div><div>On Oct 6, 2011, at 2:54 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Thu, Oct 6, 2011 at 1:27 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Author: bwilson<br>
Date: Thu Oct 6 15:27:40 2011<br>
New Revision: 141309<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=141309&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=141309&view=rev</a><br>
Log:<br>
Rearrange for readability. No functional change.<br>
<br>
Modified:<br>
cfe/trunk/lib/Driver/ToolChains.cpp<br>
<br>
Modified: cfe/trunk/lib/Driver/ToolChains.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=141309&r1=141308&r2=141309&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=141309&r1=141308&r2=141309&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/lib/Driver/ToolChains.cpp (original)<br>
+++ cfe/trunk/lib/Driver/ToolChains.cpp Thu Oct 6 15:27:40 2011<br>
@@ -131,9 +131,10 @@<br>
if (Value == "armv4t")<br>
return "armv4t";<br>
<br>
- if (Value == "armv7" || Value == "armv7-a" || Value == "armv7-r" ||<br>
- Value == "armv7-m" || Value == "armv7a" || Value == "armv7r" ||<br>
- Value == "armv7m")<br>
+ if (Value == "armv7" ||<br>
+ Value == "armv7a" || Value == "armv7-a" ||<br>
+ Value == "armv7r" || Value == "armv7-r" ||<br>
+ Value == "armv7m" || Value == "armv7-m")<br></blockquote><div><br></div><div>If you're cleaning in here, use StringSwitch?</div></div>
</blockquote></div><br></div></body></html>