[llvm] r293373 - This addresses LLDB bug 31699, which was caused by LLVM using static linking on Windows.

[Vadim Chugunov via llvm-commits]

>
>
> I tried upgrading to CMake 3.7 and re-buliding but that didn't help :-/
>
> I think I have the SDK; at least I have a "v10.0A" dir in C:\Program
> Files (x86)\Microsoft SDKs\Windows.


Can you please check whether your SDK has the "Redist\ucrt\DLLs" subfolder?
 (for example, Windows SDK shipped with VS2015 did not have it).  You may
need to grab latest SDK from here
<https://developer.microsoft.com/en-us/windows/downloads/windows-10-sdk>.


> Shouldn't cmake error out or something if it can't find the files?
>

Unfortunately it does not.  (If you are using cmake-gui, it'll show the
redist path variable in red, but this isn't a hard error).


> > Also, looks like you need to update the download url to
> > LLVM-**5**.0.0-r293474-win32.exe
>
> Fixed. Thanks for noticing!
>
>
> > On Mon, Jan 30, 2017 at 7:01 PM, Hans Wennborg <hans at chromium.org>
> wrote:
> >>
> >> I've just built a new snapshot that includes this revision:
> >> http://llvm.org/pre-releases/win-snapshots/LLVM-4.0.0-r293474-win32.exe
> >>
> >> Can you please verify that it works and includes the correct DLLs? And
> >> does it help LLDB?
> >>
> >> When I install it, I see these in the bin\ directory:
> >> msvcp140.dll
> >> vcruntime140.dll
> >>
> >> Thanks for pushing this forward!
> >>
> >> Hans
> >>
> >>
> >> On Fri, Jan 27, 2017 at 11:39 PM, Vadim Chugunov via llvm-commits
> >> <llvm-commits at lists.llvm.org> wrote:
> >> > Author: vadimcn
> >> > Date: Sat Jan 28 01:39:52 2017
> >> > New Revision: 293373
> >> >
> >> > URL: http://llvm.org/viewvc/llvm-project?rev=293373&view=rev
> >> > Log:
> >> > This addresses LLDB bug 31699, which was caused by LLVM using static
> >> > linking on Windows.
> >> >
> >> > In order to make sure that LLVM continues to work on machines that do
> >> > not have the Universal CRT yet,
> >> > we'll need to ship a copy of UCRT in the Windows installation package.
> >> > Fortunately, CMake 3.6+ already
> >> > supports app-local deployment of UCRT dlls, we just need to turn this
> >> > on.
> >> >
> >> > Differential Revision: https://reviews.llvm.org/D29146
> >
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170131/ba3d867c/attachment.html>