[Lldb-commits] MSVC12 support

Virgile Bello virgile.bello at gmail.com
Wed Aug 28 06:53:06 PDT 2013


Hello,

Here is the tentative patch for MSVC12 support. Actually much smaller than
MinGW32 one (1/3).
Need some improvements/reviews. Some points that might need some discussion:

* getopt.h doesn't exist. Also, current inttypes.h doesn't work well
(should be fixed in final release so it will be removed later).
As a result, I added a folder include/msvc that can override some system
include.
I wanted to make sure it was OK to do so? If yes, is this folder OK or
somewhere else maybe?
Otherwise, I could always add lot of #ifdef around each <getopt.h> include,
but that might be ugly.
Note that MSVC11 might also need some more additional includes as well
(need to check).

* Most of the other fixes are related to some subtle difference in
templates instantiation, as well as friend class within namespace that
needs forward declaration (cf CommandObjectBreakpointCommandAdd
and CommandObjectProcessLaunch)..

* Also, I added backslash separator for paths in FileSpec.cpp (apply for
MinGW32 as well).
I might commit this separately if the MSVC patch takes some time to be
cleared out.

* Only thing left to fix before it compiles is a MinGW32/MSVC compilation
error introduced yesterday by r189316 (otherwise it would probably have
been enough).
It's due to windows.h and llvm/Support/COFF.h conflicting #define/enum. I
hope to get this fixed in LLVM soon too.

* I think the only thing I didn't include is a few printf("%zx") in logging
that needs to be fixed here and there (will be in a separate patch).

Compilation OK on Linux.

Thanks,
Virgile
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20130828/fffa9161/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lldb-msvc12.diff
Type: application/octet-stream
Size: 38492 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20130828/fffa9161/attachment.obj>


More information about the lldb-commits mailing list