Hello, Kim.<div><br></div><div>Thank you to work on Win32.</div><div><br></div><div>Could you use llvm::sys::PathSeparator there? Please confirm in your environment.</div><div><br></div><div>FYI, I heard llvm::sys might be deprecated in future. That said, I am not sure when it done.</div>
<div>We may use llvm::sys for now.</div><div><br></div><div><div>--- lib/Driver/Driver.cpp<span class="" style="white-space:pre">   </span>(revision 169719)</div><div>+++ lib/Driver/Driver.cpp<span class="" style="white-space:pre"> </span>(working copy)</div>
<div>@@ -238,7 +238,11 @@</div><div>   if (char *env = ::getenv("COMPILER_PATH")) {</div><div>     StringRef CompilerPath = env;</div><div>     while (!CompilerPath.empty()) {</div><div>-      std::pair<StringRef, StringRef> Split = CompilerPath.split(':');</div>
<div>+#ifdef LLVM_ON_WIN32</div><div>+    std::pair<StringRef, StringRef> Split = CompilerPath.split(';');</div><div>+#else</div><div>+    std::pair<StringRef, StringRef> Split = CompilerPath.split(':');</div>
<div>+#endif</div><div></div></div><div><br></div><div>Please be aware of indentation. We prefer to follow the style in existing code.</div><div><a href="http://llvm.org/docs/CodingStandards.html#golden-rule">http://llvm.org/docs/CodingStandards.html#golden-rule</a></div>
<div><br></div><div>...Takumi</div><div><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2012/12/11 김보곤 <span dir="ltr"><<a href="mailto:bogon82.kim@samsung.com" target="_blank">bogon82.kim@samsung.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div>
<p>Hello, everyone.</p>
<p> </p>
<p>The COMPILER_PATH environment seperator is semicolon on windows as colon on linux</p>
<p> </p>
<p>To support multiple path of COMPILER_PATH env, change the seperator to semicolon for windows platform</p>
<p> </p>
<p>P.S</p>
<p>This is my first contribution. Please let me know if there is any mistakes</p>
<p> </p>
<table style="width:550px;font-size:10pt" border="0" cellspacing="0" cellpadding="0" width="550">
<tbody>
<tr>
<td height="20" width="261">
<table style="line-height:150%;font-family:굴림체,arial;font-size:9pt" border="0" cellspacing="0" cellpadding="10" width="421" bgcolor="#ffffff">
<tbody>
<tr>
<td style="line-height:150%;font-family:굴림체,arial;font-size:9pt" height="113" width="159">
<p><img width="156" height="56"></p>
<p align="center"><font color="navy"><b></b></font> </p></td>
<td style="line-height:70%;font-family:굴림체,arial;font-size:9pt" height="113" width="222">
<p><font color="navy"><b><span style="font-size:11pt">김 보 곤 (Bogon Kim)</span></b></font> </p>
<p><span style="font-size:12pt"></span> </p>
<p><font color="navy"><b>Platform R&D Group</b></font></p>
<p><font color="navy"><b>Mobile Communication Division</b></font></p>
<p><font color="navy"><b>Samsung Electronics.Co.,LTD</b></font></p>
<p><font face="Arial"><b>TEL       82 - 31 - 301 - 2814</b></font></p>
<p><font face="Arial"><b>Mobile  82 - 11 - 881 - 8458</b></font></p>
<p><font face="Arial"><b>E-mail  <a href="mailto:bogon82.kim@samsung.com" target="_blank">bogon82.kim@samsung.com</a></b></font></p>
<p><strong><font color="#800080" face="Arial">With passion and confidence(WPAC)</font></strong></p></td></tr></tbody></table>
<p> </p></td>
<td height="20" width="283">
<p> </p>
<p> </p>
<p> </p>
<p> </p>
<p> </p>
<p> </p>
<p> </p>
<p> </p></td></tr></tbody></table>
<p> </p>
<p> </p>
<table>
<tbody>
<tr>
<td>
<p><img border="0" src="cid:PYMC4CBBEM6S@namo.co.kr" width="520"></p></td></tr></tbody></table></div><img border="0" width="0" height="0"><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>