While trying to say that my patch is not really needed I failed to explain the main advantage. We'll remove a bunch of code that deals with windows registry to detect installed version of visual studio and sdk. Right now Clang prefers the version of the toolchain that was used to compile it, which makes little sense for people using multiple versions of VS. It will also allow you to target x64 from 32-bit Clang without having to specify the target triple, you will just run clang from the cross tools command prompt or run the appropriate vcvars.bat.<div>

<br></div><div>Hope this helps :)<br><br><div class="gmail_quote">On Tue, Jun 12, 2012 at 2:47 PM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@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 Tue, Jun 12, 2012 at 2:26 PM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>> wrote:<br>


> It will search the path to find the first link.exe. The one from Visual<br>
> Studio should be first if Command prompt or vcvars.bat are used.<br>
<br>
</div>Oh, OK. But then I don't see what you're intending to patch. It already works:<br>
<br>
--<br>
[14:45:19,41] D:\dev\llvm-trunk\build\bin\Debug<br>
>"C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\vcvarsall.bat"<br>
Setting environment for using Microsoft Visual Studio 2010 x86 tools.<br>
<br>
[14:45:57,55] D:\dev\llvm-trunk\build\bin\Debug<br>
>clang t.c<br>
<br>
[14:46:02,01] D:\dev\llvm-trunk\build\bin\Debug<br>
>a.out<br>
Hello, worldliness!<br>
<br>
[14:46:03,95] D:\dev\llvm-trunk\build\bin\Debug<br>
><br>
--<br>
<br>
Curious,<br>
- Kim<br>
</blockquote></div><br></div>