[Lldb-commits] [lldb] r334861 - Fix TestExec after r334783

Frederic Riss via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 15 13:36:03 PDT 2018


Author: friss
Date: Fri Jun 15 13:36:03 2018
New Revision: 334861

URL: http://llvm.org/viewvc/llvm-project?rev=334861&view=rev
Log:
Fix TestExec after r334783

The second makefile that was added has implicit rules which meant
that secondprog.cpp would be built once into a secondprog binary,
but it would also be compiled as a.out overwriting the main binary.
This lead to spurious failures.

This commit simplifies the Makefile to build only once with the correct
executable name.

Modified:
    lldb/trunk/packages/Python/lldbsuite/test/functionalities/exec/secondprog.mk

Modified: lldb/trunk/packages/Python/lldbsuite/test/functionalities/exec/secondprog.mk
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/exec/secondprog.mk?rev=334861&r1=334860&r2=334861&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/functionalities/exec/secondprog.mk (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/exec/secondprog.mk Fri Jun 15 13:36:03 2018
@@ -1,13 +1,6 @@
 LEVEL = ../../make
 
 CXX_SOURCES := secondprog.cpp
-
-all: secondprog
-
-secondprog:
-	$(CXX) $(CXXFLAGS) -o secondprog $(SRCDIR)/secondprog.cpp
-
-clean::
-	rm -rf secondprog secondprog.dSYM
+EXE = secondprog
 
 include $(LEVEL)/Makefile.rules




More information about the lldb-commits mailing list