cameron314 added a comment.

Hmm, that doesn't seem good. Locally it links for me (I get compile time 
warnings about the signal stuff, though).
It looks like the WinSDK signal.h is being pulled in despite `_INC_SIGNAL` 
being defined (note that there's no include guard in that header -- it uses 
`#pragma once` instead). And the signal handler callback has a different return 
type. I don't see anything related to Unicode.

Here's the tail of my build output for lldb-mi.exe. I'm using VS2015 on Windows 
7 x64, compiling in release with ninja:

  [77/78] 13.653s Building CXX object 
tools\lldb\tools\lldb-mi\CMakeFiles\lldb-mi.dir\Platform.cpp.obj
  d:\dev\llvm\tools\lldb\tools\lldb-mi\Platform.h(84): warning C4005: 
'SIG_DFL': macro redefinition
  C:\Program Files (x86)\Windows 
Kits\10\include\10.0.10240.0\ucrt\signal.h(35): note: see previous definition 
of 'SIG_DFL'
  d:\dev\llvm\tools\lldb\tools\lldb-mi\Platform.h(85): warning C4005: 
'SIG_IGN': macro redefinition
  C:\Program Files (x86)\Windows 
Kits\10\include\10.0.10240.0\ucrt\signal.h(36): note: see previous definition 
of 'SIG_IGN'
  d:\dev\llvm\tools\lldb\tools\lldb-mi\Platform.h(87): warning C4273: 'signal': 
inconsistent dll linkage
  C:\Program Files (x86)\Windows 
Kits\10\include\10.0.10240.0\ucrt\signal.h(57): note: see previous definition 
of 'signal'
  
  [78/78] 14.332s Linking CXX executable bin\lldb-mi.exe
     Creating library lib\lldb-mi.lib and object lib\lldb-mi.exp


http://reviews.llvm.org/D17107



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to