We do want to support 2015, but the idea is that the batch file is not actually even used anymore after this patch (outside of development anyway).<br><br>The vsix installs into 2015 just fine <br><div class="gmail_quote"><div dir="ltr">On Thu, Feb 1, 2018 at 2:31 AM Hans Wennborg via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hans added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/tools/msbuild/install.bat:9<br>
<br>
 REM Older versions of VS would look for these files in the Program Files\MSBuild directory<br>
 REM but with VS2017 it seems to look for these directly in the Visual Studio instance.<br>
----------------<br>
Oh I see, this is based on top off D42762 which isn't landed yet. I'll review that first then :-)<br>
<br>
<br>
================<br>
Comment at: llvm/tools/msbuild/uninstall.bat:8<br>
<br>
-set PLATFORM=None<br>
-:LOOPHEAD<br>
-IF %PLATFORM% == x64 GOTO LOOPEND<br>
-IF %PLATFORM% == Win32 SET PLATFORM=x64<br>
-IF %PLATFORM% == None SET PLATFORM=Win32<br>
+SET VCTargets=%ProgramFiles(x86)%\Microsoft Visual Studio\2017\Professional\Common7\IDE\VC\VCTargets<br>
<br>
----------------<br>
I thought we also wanted to support VS 2015?<br>
<br>
<br>
================<br>
Comment at: llvm/tools/msbuild/uninstall.bat:15<br>
+ECHO Uninstalling x64 Platform Toolset<br>
+SET PlatformToolsets=%VCTargets%\Platforms\x64\PlatformToolsets<br>
+IF EXIST "%PlatformToolsets%\llvm\Toolset.props" del "%PlatformToolsets%\llvm\Toolset.props"<br>
----------------<br>
Does this need quotes? Same for the SET in the next section.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D42769" rel="noreferrer" target="_blank">https://reviews.llvm.org/D42769</a><br>
<br>
<br>
<br>
</blockquote></div>