<div dir="ltr">Those weren't actually used, so I put something there to suppress warnings generated by the build system (those values get used elsewhere for somethings that's not important to clang).  I've updated the patch to suppress the warnings more explicitly.<div>
<br></div><div>I changed the path from bin to msbuild-bin</div><div><br></div><div>The fixed registry key location is left as a TODO.  We probably want to have the build system create this file with the current version number and have the installer also use the current version number to select the destination folder.</div>
<div><br></div><div>-Warren</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 27, 2013 at 3:26 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.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, Aug 27, 2013 at 6:16 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>

> It seems reasonable for these to have DOS line endings.  Do they need to<br>
> have a BOM, though?<br>
><br>
> +    <ClangInstallDir>$(Registry:HKEY_LOCAL_MACHINE\SOFTWARE\LLVM\LLVM<br>
> 3.4.svn)</ClangInstallDir><br>
><br>
> Should this be looking for "LLVM 3.4.svn" specifically, or is there a way to<br>
> be more generic?<br>
><br>
> +<br>
> <ExecutablePath>$(ClangInstallDir)\bin;$(ExecutablePath)</ExecutablePath><br>
><br>
> We should probably add a new directory like msbuild-bin instead, since we<br>
> really don't want to place a clang binary named cl.exe into bin/.<br>
><br>
><br>
><br>
> On Tue, Aug 27, 2013 at 3:05 PM, Warren Hunt <<a href="mailto:whunt@google.com">whunt@google.com</a>> wrote:<br>
>><br>
>> Attached is a patch with the requisite files to integrate a<br>
>> drop-selectable llvm tool-chain in Visual Studio.  The files need to be<br>
>> copied to:<br>
>><br>
>> C:\Program Files<br>
>> (x86)\MSBuild\Microsoft.Cpp\v4.0\Platforms\Win32\PlatformToolsets\llvm<br>
>> on a 64 bit machine<br>
>><br>
>> or<br>
>><br>
>> C:\Program<br>
>> Files\MSBuild\Microsoft.Cpp\v4.0\Platforms\Win32\PlatformToolsets\llvm<br>
>> on a 32 bit machine<br>
>><br>
>> or wherever MSBuild is installed.  This information can be obtained from<br>
>> the registry if desired.<br>
<br>
</div>Would it make sense for us to add something to automatically perform<br>
the copy for the user?<br>
<br>
> +    <WindowsSdkDir Condition="'$(UseEnv)' != 'true'">$(Registry:HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v7.0A@InstallationFolder)</WindowsSdkDir><br>
> +    <WindowsSdkDir Condition="'$(WindowsSdkDir)' == ''">$(Registry:HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\Microsoft SDKs\Windows\v7.0A@InstallationFolder)</WindowsSdkDir><br>

<br>
What about SDK dirs other than 7.0A?<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>