[PATCH] utils/llvm-lit: Generate "llvm-lit.py" rather than "llvm-lit" on Win32 hosts to let llvm-lit.py --use-processes work.

NAKAMURA Takumi geek4civic at gmail.com
Mon Dec 2 23:26:55 PST 2013


Hi ddunbar,

llvm-lit needs suffix.py for multiprocess to find a main module with Python win32.

http://llvm-reviews.chandlerc.com/D2313

Files:
  llvm/utils/llvm-lit/CMakeLists.txt
  llvm/utils/llvm-lit/Makefile

Index: llvm/utils/llvm-lit/CMakeLists.txt
===================================================================
--- llvm/utils/llvm-lit/CMakeLists.txt
+++ llvm/utils/llvm-lit/CMakeLists.txt
@@ -1,4 +1,9 @@
+if (WIN32 AND NOT CYGWIN)
+  # llvm-lit needs suffix.py for multiprocess to find a main module.
+  set(suffix .py)
+endif ()
+
 configure_file(
   llvm-lit.in
-  ${LLVM_TOOLS_BINARY_DIR}/llvm-lit
+  ${LLVM_TOOLS_BINARY_DIR}/llvm-lit${suffix}
   )
Index: llvm/utils/llvm-lit/Makefile
===================================================================
--- llvm/utils/llvm-lit/Makefile
+++ llvm/utils/llvm-lit/Makefile
@@ -11,9 +11,14 @@
 
 include $(LEVEL)/Makefile.common
 
-all:: $(ToolDir)/llvm-lit
+# llvm-lit needs suffix.py for multiprocess to find a main module.
+ifeq ($(HOST_OS),MingW)
+  Suffix := .py
+endif
 
-$(ToolDir)/llvm-lit: llvm-lit.in Makefile $(ToolDir)/.dir
+all:: $(ToolDir)/llvm-lit$(Suffix)
+
+$(ToolDir)/llvm-lit$(Suffix): llvm-lit.in Makefile $(ToolDir)/.dir
 	$(Echo) "Creating 'llvm-lit' script..."
 	$(Verb)$(ECHOPATH) s=@LLVM_SOURCE_DIR@=$(LLVM_SRC_ROOT)=g > lit.tmp
 	$(Verb)$(ECHOPATH) s=@LLVM_BINARY_DIR@=$(LLVM_OBJ_ROOT)=g >> lit.tmp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2313.1.patch
Type: text/x-patch
Size: 1173 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131202/2c3b53cf/attachment.bin>


More information about the llvm-commits mailing list