Unfortunately the new integration will *not* work in vs2015.  I meant to fix this in the vsix (that was the other change I forgot to push).  The reason is that the ability of an extension to properly install itself as a toolchain only became possible in 2017, so there’s no way that I’m aware to even do this in 2015 with a vsix<br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 8, 2018 at 1:41 PM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Aug 8, 2018 at 5:39 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
> I can theorize that perhaps what breaks is that if you have an old VS<br>
> integration + LLVM installed, then install a new LLVM, the old VS<br>
> integration will not be able to find the new clang-cl.exe.<br>
<br>
The old VS integration would also have been uninstalled in that case.<br>
<br>
> I don't have a<br>
> good idea on how to address this off the top of my head except perhaps to<br>
> create the msbuild-bin folder but put a DEPRECATED.TXT file in there so<br>
> people will find it.<br>
><br>
> The other problem though is that the new VS integration only works with >=<br>
> 2017, so if someone can't upgrade for some reason, this would still be a<br>
> problem for them.  Perhaps we could continue copying the main executable<br>
> there for backwards compatibility?<br>
<br>
We'd need to keep installing the old integrations also, and I don't<br>
think we want to keep them around. I think the new extension also<br>
supports VS 2015, at least it installs for it but I didn't actually<br>
try the integration there. Supporting 2017 and 2015 is probably<br>
enough.<br>
<br>
Let's hear from degski what his reason for needing msbuild-bin/cl.exe was.<br>
<br>
Maybe we should just put a bigger note on the download page that the<br>
old integration is gone and people need to get the new one.<br>
<br>
><br>
> On Wed, Aug 8, 2018 at 6:50 AM Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
>><br>
>> As noted on <a href="http://llvm.org/builds/" rel="noreferrer" target="_blank">http://llvm.org/builds/</a> the installer no longer includes<br>
>> the Visual Studio integration, instead a new one can be downloaded<br>
>> from<br>
>> <a href="https://marketplace.visualstudio.com/items?itemName=LLVMExtensions.llvm-toolchain" rel="noreferrer" target="_blank">https://marketplace.visualstudio.com/items?itemName=LLVMExtensions.llvm-toolchain</a><br>
>> and that does not rely on having the "fake" cl.exe executable.<br>
>><br>
>> Can you share more details on what broke because the installer doesn't<br>
>> include msbuild-bin/cl.exe? This will also apply to the upcoming llvm<br>
>> 7 release, so it would be good to understand if this causes problems.<br>
>><br>
>> Thanks,<br>
>> Hans<br>
>><br>
>> On Wed, Aug 8, 2018 at 7:29 AM, degski via cfe-dev<br>
>> <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br>
>> > If anyone feels this email does not concern them then please ignore it.<br>
>> ><br>
>> > Both LLVM-8.0.0-r339039-win64.exe and LLVM-7.0.0-rc1-win64.exe do not<br>
>> > create<br>
>> > the folder ..\LLVM\msbuild-bin and a copy of the main executable renamed<br>
>> > to<br>
>> > cl.exe in that folder. Mahem ensues.<br>
>> ><br>
>> > I'm not writing this for my own purpose, I've solved the issue but for<br>
>> > those<br>
>> > after me.<br>
>> ><br>
>> > Have a nice day,<br>
>> ><br>
>> > degski<br>
>> > --<br>
>> > "If something cannot go on forever, it will stop" - Herbert Stein<br>
>> ><br>
>> > _______________________________________________<br>
>> > cfe-dev mailing list<br>
>> > <a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
>> ><br>
</blockquote></div>