[Lldb-commits] [PATCH] Fix makefiles to build shared libraries (DLLs) for tests on Windows

Adrian McCarthy amccarth at google.com
Fri Mar 13 13:53:02 PDT 2015

Comment at: test/make/Makefile.rules:378
@@ -375,1 +377,3 @@
zturner wrote:
> I think this should be called something like COMPILING_LLDB_TEST_DLL or COMPILING_TEST_DLL.  Or anything as long as it has "DLL" in it.  Because an EXE is technically an "LLDB Test" also, but it wouldn't have this definition.

Comment at: test/make/Makefile.rules:413
@@ +412,3 @@
+ifeq "$(OS)" "Windows_NT"
+	QUOTE = "
zturner wrote:
> I'd probably change the name from SEMICOLON to something like JOIN_COMMAND.  It's a little confusing to see SEMICOLON = &.  
Agreed and done.

Comment at: test/make/test_common.h:19
@@ +18,3 @@
+// force include this header file.
+static void *__uncaught_exception() { return nullptr; }
zturner wrote:
> I know you were just copying over my code, but my code was actually wrong here :)   __uncaught_exception() should return bool.  Can you change this to an inline bool which returns true?  https://msdn.microsoft.com/en-us/library/ff770584.aspx?f=255&MSPPError=-2147217396  This is probably only of theoretical value, but we might as well.
I've changed this to #include <eh.h> and rewritten the comment to describe the better understand we arrived at last night.



More information about the lldb-commits mailing list