[Testsuite] Get rid of most of the recursive shared library Makefiles

[friss]

Summary

I was working on cleaning up our on-device test runs last week, and as
a result I needed to update a lot of test Makefiles in the Swift
tree. I found it especailly annoying to update the build system of
tests that are building mutliple images (typically a shared
library).

The main reason is that you can't see what's (if anything) special
about the build from the main makefile. You always need to open
mutliple Makefiles. Another reason is the way you have to add a
special line to your clean rule to do the right thing, and you never
really know which arguments to pass.

Most of the secondary Makefiles we have are just a couple variable
definitions and then an include of Makefile.rules. This patch removes
most of the secondary Makefiles and replaces them with a direct
invocation of Makefile.rules in the main Makefile. The specificities
of each sub-build are listed right there on the recursive $(MAKE)
call. All the variables that matter are being passed automagically by
make as they have been passed on the command line. The only things you
need to specify are the variables customizating the Makefile.rules
logic for each image.

This patch also deals with the clean rules using this trick to avoid
having to duplicate the recursive invocation in a separate clean rule:

mylib clean::

$(MAKE) -f $(MAKEFILE_RULES) ... $(MAKECMDGOALS)

The MAKECMDGOALS variable at the end contains the list of build goals
that have been passed on the command line. When it's empty, this
invocation just calls the default Makefile.rules rule which is what
you want. When "clean" is passed as a goal, the rule will be invoked
too and it is passed through MAKECMDGOALS invoking the appropriate
cleaning logic from Makefile.rules.

I really like the conciseness of this approach and the fact that
everything is visible in one place. What do others think?

[labath]

I like the idea of passing the variable values "inline". The MAKECMDGOALS thingy seems... clever. I sort of like it, and it also feels like it might be too much. I suppose that is fine, but.. do we actually need a functional clean rule? I was under the impression that ever since we moved tests to build out-of-tree the clean rule doesn't do much and isn't even being invoked by default. Should we just delete the clean rules altogether? The handful of tests that actually want to clean and rebuild as a part of the test can do something custom...

[aprantl]

This is some really tricky Make (wow), but I think the benefit of it being so much shorter and localized outweighs the steeper learning curve. Thanks!

lldb/packages/Python/lldbsuite/test/commands/expression/top-level/Makefile

9

Can you make sure that this is passing what's necessary by removing the default rule from Makefile.rules that initializes CC ?=

lldb/packages/Python/lldbsuite/test/commands/target/create-deps/Makefile

1

= -L$(BUILDDIR) ?
Perhaps := -L. is preferable.

lldb/packages/Python/lldbsuite/test/lang/cpp/namespace_definitions/Makefile

1

Is $(LIBPREFIX) available here? shouldn't this be = instead?

lldb/packages/Python/lldbsuite/test/lang/objc/conflicting-definition/Makefile

1

:= and $(BUILDDIR)

2

:=

8

Let's move these past the include Makefile.rules for symmetry.

[friss]

In D68558#1697890, @labath wrote:

I like the idea of passing the variable values "inline". The MAKECMDGOALS thingy seems... clever. I sort of like it, and it also feels like it might be too much.

This is why I didn't commit this without review, I do feel the same.

do we actually need a functional clean rule? I was under the impression that ever since we moved tests to build out-of-tree the clean rule doesn't do much and isn't even being invoked by default. Should we just delete the clean rules altogether? The handful of tests that actually want to clean and rebuild as a part of the test can do something custom...

Pretty sure the test runs don't need the clean rule, but it's nice to have when manually iterating over the build phase of a test. OTOH, manually doing a "rm -rf" of the build directory is not a big burden.

Anybody else objects to getting rid of the clean rules?

[JDevlieghere]

In D68558#1698031, @friss wrote:

Pretty sure the test runs don't need the clean rule, but it's nice to have when manually iterating over the build phase of a test. OTOH, manually doing a "rm -rf" of the build directory is not a big burden.

Anybody else objects to getting rid of the clean rules?

I like having the clean rule when I'm debugging a particular test and I'm messing with the Makefile. I don't mind having to manually remove the build directory, but it seems like we could just make that the top-level clean rule. If we go that route, let's put in a safeguard so that you don't accidentally remove your current working directory if you forget to pass the -C flag.

[friss]

Addressed review feedback.

Completely removed the clean code for the Makefiles and reworked it in
Makefile.rules to be much simpler and catch-all.

[aprantl]

lldb/packages/Python/lldbsuite/test/make/Makefile.rules

790

I would just error, period. IMHO make clean shouldn'd delete anything that make doesn't know how to rebuild and I don't think we have any rule to restore $(BUILDDIR), do we?

[friss]

lldb/packages/Python/lldbsuite/test/make/Makefile.rules

790

We don't delete $(BUILDDIR), we delete what's inside of it. I would have gotten rid of the rule completely, but having a clean rule is handy in one circumstance: when iterating over the build of a new test (or like I did last week rework a lot of existing test builds). There you want to be able to run the make command by itself, and being able to quickly iterate with make clean all is really nice. I don't think we're taking any risk here as the rule checks whether we are inside a directory that dotest is going to wipe anyway.

[aprantl]

We don't delete $(BUILDDIR), we delete what's inside of it.

Sorry, I missed that detail!

[JDevlieghere]

Thanks, LGTM!

lldb/packages/Python/lldbsuite/test/commands/expression/top-level/Makefile

 CXX_SOURCES := main.cpp test.cpp
 
-include Makefile.rules
+all: dummy
 
-a.out: dummy
+include Makefile.rules
 
-dummy:
-  $(MAKE) VPATH=$(VPATH) -I $(SRCDIR) -f $(SRCDIR)/dummy.mk
+dummy clean:: dummy.cpp
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    CXX_SOURCES=dummy.cpp EXE=dummy $(MAKECMDGOALS)
 
-clean::
-  $(MAKE) VPATH=$(VPATH) -I $(SRCDIR) -f $(SRCDIR)/dummy.mk clean

lldb/packages/Python/lldbsuite/test/commands/expression/top-level/dummy.mk

-CXX_SOURCES := dummy.cpp
-EXE := dummy
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/commands/target/create-deps/Makefile

-LIB_PREFIX := load_
-
-LD_EXTRAS := -L. -l$(LIB_PREFIX)a
+LD_EXTRAS := -L. -lload_a
 CXX_SOURCES := main.cpp
 
-include Makefile.rules
-
-a.out: lib_a
+a.out: libload_a
 
-lib_%:
-  $(MAKE) VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/$*.mk
+include Makefile.rules
 
-clean::
-  $(MAKE) -f $(SRCDIR)/a.mk clean
+libload_a clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_NAME=load_a DYLIB_CXX_SOURCES=a.cpp $(MAKECMDGOALS)

lldb/packages/Python/lldbsuite/test/commands/target/create-deps/a.mk

-LIB_PREFIX := load_
-
-DYLIB_NAME := $(LIB_PREFIX)a
-DYLIB_CXX_SOURCES := a.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/functionalities/exec/Makefile

 CXX_SOURCES := main.cpp
 
-all: a.out secondprog
+all: secondprog
 
 include Makefile.rules
 
-secondprog:
-  $(MAKE) VPATH=$(VPATH) -f $(SRCDIR)/secondprog.mk
-
-clean::
-  $(MAKE) -f $(SRCDIR)/secondprog.mk clean
+secondprog clean:: secondprog.cpp
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    CXX_SOURCES=secondprog.cpp EXE=secondprog $(MAKECMDGOALS)

lldb/packages/Python/lldbsuite/test/functionalities/exec/secondprog.mk

-CXX_SOURCES := secondprog.cpp
-EXE = secondprog
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/functionalities/jitloader_gdb/Makefile

 C_SOURCES := main.c
 
 all: a.out simple
 
 include Makefile.rules
 
-simple:
-  $(MAKE) VPATH=$(VPATH) -f $(SRCDIR)/simple.mk
-
-clean::
-  $(MAKE) -f $(SRCDIR)/simple.mk clean
+simple clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    C_SOURCES=simple.c EXE=simple $(MAKECMDGOALS)
 

lldb/packages/Python/lldbsuite/test/functionalities/jitloader_gdb/simple.mk

-C_SOURCES := simple.c
-EXE = simple
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/functionalities/load_unload/Makefile

-LIB_PREFIX := loadunload_
-
-LD_EXTRAS := -L. -l$(LIB_PREFIX)d
+LD_EXTRAS := -L. -lloadunload_d
 CXX_SOURCES := main.cpp
 USE_LIBDL := 1
 
+a.out: lib_b lib_a lib_c lib_d hidden_lib_d
+
 include Makefile.rules
 
-a.out: lib_a lib_b lib_c lib_d hidden_lib_d install_name_tool
+lib_a:: lib_b
 
-lib_%:
-  $(MAKE) VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/$*.mk
+lib_a clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_CXX_SOURCES=a.cpp DYLIB_NAME=loadunload_a \
+    LD_EXTRAS="-L. -lloadunload_b" $(MAKECMDGOALS)
+
+lib_b clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_CXX_SOURCES=b.cpp DYLIB_NAME=loadunload_b $(MAKECMDGOALS)
+
+lib_c clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_CXX_SOURCES=c.cpp DYLIB_NAME=loadunload_c $(MAKECMDGOALS)
+
+lib_d clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_CXX_SOURCES=d.cpp DYLIB_NAME=loadunload_d $(MAKECMDGOALS)
 
-install_name_tool:
 ifeq ($(OS),Darwin)
+lib_d::
   install_name_tool -id @executable_path/libloadunload_d.dylib libloadunload_d.dylib
 endif
 
-
-hidden_lib_d:
-  $(MAKE) VPATH=$(SRCDIR)/hidden -I $(SRCDIR)/hidden -C hidden -f $(SRCDIR)/hidden/Makefile
-
-clean::
-  $(MAKE) -f $(SRCDIR)/a.mk clean
-  $(MAKE) -f $(SRCDIR)/b.mk clean
-  $(MAKE) -f $(SRCDIR)/c.mk clean
-  $(MAKE) -f $(SRCDIR)/d.mk clean
-  $(MAKE) -I $(SRCDIR)/hidden -C hidden -f $(SRCDIR)/hidden/Makefile clean
+hidden_lib_d clean::
+  $(MAKE) VPATH=$(SRCDIR)/hidden -C hidden -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_CXX_SOURCES=d.cpp DYLIB_NAME=loadunload_d $(MAKECMDGOALS)

lldb/packages/Python/lldbsuite/test/functionalities/load_unload/a.mk

-LIB_PREFIX := loadunload_
-
-LD_EXTRAS := -L. -l$(LIB_PREFIX)b
-
-DYLIB_NAME := $(LIB_PREFIX)a
-DYLIB_CXX_SOURCES := a.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules
-
-$(DYLIB_FILENAME): lib_b
-
-.PHONY lib_b:
-  $(MAKE) VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/b.mk
-
-clean::
-  $(MAKE) -I $(SRCDIR) -f $(SRCDIR)/b.mk clean

lldb/packages/Python/lldbsuite/test/functionalities/load_unload/b.mk

-LIB_PREFIX := loadunload_
-
-DYLIB_NAME := $(LIB_PREFIX)b
-DYLIB_CXX_SOURCES := b.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/functionalities/load_unload/c.mk

-LIB_PREFIX := loadunload_
-
-DYLIB_NAME := $(LIB_PREFIX)c
-DYLIB_CXX_SOURCES := c.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/functionalities/load_unload/d.mk

-LIB_PREFIX := loadunload_
-
-DYLIB_EXECUTABLE_PATH := $(CURDIR)
-
-DYLIB_NAME := $(LIB_PREFIX)d
-DYLIB_CXX_SOURCES := d.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/functionalities/load_using_paths/Makefile

 CXX_SOURCES := main.cpp
 USE_LIBDL := 1
 
-include Makefile.rules
-
 all: hidden_lib a.out
 
-hidden_lib:
-  $(MAKE) VPATH=$(SRCDIR)/hidden -I $(SRCDIR)/hidden -C hidden -f $(SRCDIR)/hidden/Makefile
+include Makefile.rules
+
+hidden_lib clean::
+  $(MAKE) VPATH=$(SRCDIR)/hidden -C hidden -f $(MAKEFILE_RULES) \
+  DYLIB_ONLY=YES DYLIB_CXX_SOURCES=d.cpp DYLIB_NAME=loadunload
 
-clean::
-  $(MAKE) -I $(SRCDIR)/hidden -C hidden -f $(SRCDIR)/hidden/Makefile clean

lldb/packages/Python/lldbsuite/test/lang/cpp/namespace_definitions/Makefile

 LD_EXTRAS := -L. -l$(LIB_PREFIX)a -l$(LIB_PREFIX)b
 CXX_SOURCES := main.cpp
 
-include Makefile.rules
+a.out: liba libb
 
-.PHONY:
-a.out: lib_a lib_b
+include Makefile.rules
 
-lib_%:
-  $(MAKE) VPATH=$(VPATH) -f $(SRCDIR)/$*.mk
+liba clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_NAME=a DYLIB_CXX_SOURCES=a.cpp $(MAKECMDGOALS)
 
-hidden_lib_d:
-  $(MAKE) -C hidden
+libb clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_NAME=b DYLIB_CXX_SOURCES=b.cpp $(MAKECMDGOALS)
 
-clean::
-  $(MAKE) -f $(SRCDIR)/a.mk clean
-  $(MAKE) -f $(SRCDIR)/b.mk clean

lldb/packages/Python/lldbsuite/test/lang/cpp/namespace_definitions/a.mk

-DYLIB_NAME := a
-DYLIB_CXX_SOURCES := a.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/lang/cpp/namespace_definitions/b.mk

-DYLIB_NAME := b
-DYLIB_CXX_SOURCES := b.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/lang/objc/conflicting-definition/Makefile

-LD_EXTRAS = -lobjc -framework Foundation
-
-include Makefile.rules
+LD_EXTRAS = -lobjc -framework Foundation -L. -lTest -lTestExt
+OBJC_SOURCES = main.m
 
 all: a.out
 
-libTest.dylib:  Test/Test.m
+a.out: libTest.dylib libTestExt.dylib
+
+libTest.dylib clean:: Test/Test.m
   mkdir -p Test
-  $(MAKE) MAKE_DSYM=YES VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/Test/Test.mk all
+  $(MAKE) MAKE_DSYM=YES -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_NAME=Test DYLIB_OBJC_SOURCES=Test/Test.m \
+    LD_EXTRAS="-lobjc -framework Foundation" \
+    CFLAGS_EXTRAS=-I$(SRCDIR) $(MAKECMDGOALS)
 
-libTestExt.dylib: TestExt/TestExt.m
+libTestExt.dylib clean:: TestExt/TestExt.m
   mkdir -p TestExt
-  $(MAKE) MAKE_DSYM=YES VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/TestExt/TestExt.mk all
+  $(MAKE) MAKE_DSYM=YES -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_NAME=TestExt DYLIB_OBJC_SOURCES=TestExt/TestExt.m \
+    LD_EXTRAS="-lobjc -framework Foundation -lTest -L." \
+    CFLAGS_EXTRAS=-I$(SRCDIR) $(MAKECMDGOALS)
 
-a.out: main.m libTest.dylib libTestExt.dylib
-  $(CC) $(LDFLAGS) -I$(SRCDIR) -L. -lTest -lTestExt -o a.out $<
-
-clean::
-  rm -rf libTest.dylib libTestExt.dylib a.out Test.o TestExt.o libTest.dylib.dSYM libTestExt.dylib.dSYM
-  $(MAKE) MAKE_DSYM=YES VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/Test/Test.mk clean
-  $(MAKE) MAKE_DSYM=YES VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/TestExt/TestExt.mk clean
+include Makefile.rules

lldb/packages/Python/lldbsuite/test/lang/objc/conflicting-definition/Test/Test.mk

-DYLIB_NAME := Test
-DYLIB_ONLY := YES
-CFLAGS_EXTRAS = -I$(SRCDIR)/..
-LD_EXTRAS = -lobjc -framework Foundation
-
-DYLIB_OBJC_SOURCES = Test/Test.m
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/lang/objc/conflicting-definition/TestExt/TestExt.mk

-DYLIB_NAME := TestExt
-DYLIB_ONLY := YES
-CFLAGS_EXTRAS = -I$(SRCDIR)/..
-LD_EXTRAS = -L. -lTest -lobjc -framework Foundation
-
-DYLIB_OBJC_SOURCES = TestExt/TestExt.m
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/macosx/lc-note/kern-ver-str/Makefile

 MAKE_DSYM := NO
 
 C_SOURCES := main.c
 
 all: a.out create-empty-corefile
 
-create-empty-corefile:
-  $(MAKE) VPATH=$(VPATH) -f $(SRCDIR)/create-empty-corefile.mk
-
-clean::
-  $(MAKE) -f create-empty-corefile.mk clean
+create-empty-corefile clean::
+  $(MAKE) -f $(MAKEFILE_RULES) EXE=create-empty-corefile \
+    C_SOURCES=create-empty-corefile.c $(MAKECMDGOALS)
 
 include Makefile.rules

lldb/packages/Python/lldbsuite/test/macosx/lc-note/kern-ver-str/create-empty-corefile.mk

-MAKE_DSYM := NO
-
-CXX_SOURCES := create-empty-corefile.cpp
-EXE = create-empty-corefile
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/macosx/macabi/Makefile

 LEVEL = ../../make
 
 C_SOURCES := main.c
 LD_EXTRAS := -L. -lfoo
 
 TRIPLE := x86_64-apple-ios13.0-macabi
 CFLAGS_EXTRAS := -target $(TRIPLE)
 
 all: libfoo.dylib a.out
 
-lib%.dylib: %.c
-  $(MAKE) MAKE_DSYM=YES CC=$(CC) \
-    ARCH=$(ARCH) DSYMUTIL=$(DSYMUTIL) \
-    BASENAME=$(shell basename $< .c) \
-    TRIPLE=x86_64-apple-macosx10.15 SDKROOT=$(SDKROOT) \
-    VPATH=$(SRCDIR) -I $(SRCDIR) -f $(SRCDIR)/dylib.mk all
+libfoo.dylib clean:: foo.c \
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_ONLY=YES DYLIB_NAME=foo DYLIB_C_SOURCES=foo.c
 
 include $(LEVEL)/Makefile.rules

lldb/packages/Python/lldbsuite/test/macosx/macabi/dylib.mk

-LEVEL = ../../make
-DYLIB_ONLY := YES
-DYLIB_NAME := $(BASENAME)
-DYLIB_C_SOURCES := $(DYLIB_NAME).c
-
-include $(LEVEL)/Makefile.rules

lldb/packages/Python/lldbsuite/test/make/Makefile.rules

 # ARCHIVE_NAME := libfoo.a
 # ARCHIVE_C_SOURCES := a.c b.c
 
 # Uncomment line below for debugging shell commands
 # SHELL = /bin/sh -x
 
 SRCDIR := $(shell dirname $(firstword $(MAKEFILE_LIST)))
 BUILDDIR := $(shell pwd)
-THIS_FILE_DIR := $(shell dirname $(lastword $(MAKEFILE_LIST)))
+MAKEFILE_RULES := $(lastword $(MAKEFILE_LIST))
+THIS_FILE_DIR := $(shell dirname $(MAKEFILE_RULES))
 LLDB_BASE_DIR := $(THIS_FILE_DIR)/../../../../../
 
 #----------------------------------------------------------------------
 # If OS is not defined, use 'uname -s' to determine the OS name.
 #
 # uname on Windows gives "windows32" or "server version windows32", but most
 # environments standardize on "Windows_NT", so we'll make it consistent here.
 # When running tests from Visual Studio, the environment variable isn't

 ifeq "$(OS)" "Windows_NT"
 # http://llvm.org/pr24589
   IF EXIST "$(EXE)" del "$(EXE)"
   $(RM) $(wildcard *.manifest *.pdb *.ilk)
 ifneq "$(DYLIB_NAME)" ""
   $(RM) $(DYLIB_NAME).lib $(DYLIB_NAME).exp
 endif
 else
   $(RM) "$(EXE)"
 endif
 
 #----------------------------------------------------------------------
 # From http://blog.melski.net/tag/debugging-makefiles/
 #
 # Usage: make print-CC print-CXX print-LD
 #----------------------------------------------------------------------
 print-%:
   @echo '$*=$($*)'
   @echo '  origin = $(origin $*)'
   @echo '  flavor = $(flavor $*)'
   @echo '   value = $(value  $*)'
 
 ### Local Variables: ###
 ### mode:makefile ###
 ### End: ###

lldb/packages/Python/lldbsuite/test/tools/lldb-server/libraries-svr4/Makefile

 LIB_PREFIX := svr4lib
-LD_EXTRAS := -L. -l$(LIB_PREFIX)_a -l$(LIB_PREFIX)_b\"
+LD_EXTRAS := -L. -lsvr4lib_a -lsvr4lib_b\"
 CXX_SOURCES := main.cpp
 USE_LIBDL := 1
 MAKE_DSYM := NO
 
-include Makefile.rules
+a.out: svr4lib_a svr4lib_b_quote
 
-a.out: $(LIB_PREFIX)_a $(LIB_PREFIX)_b_quote
+include Makefile.rules
 
-svr4lib_%:
-  $(MAKE) VPATH=$(SRCDIR) -I $(SRCDIR) -f "$(SRCDIR)/$(LIB_PREFIX)_$*.mk"
+svr4lib_a clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_NAME=svr4lib_a DYLIB_CXX_SOURCES=svr4lib_a.cpp \
+    DYLIB_ONLY=YES $(MAKECMDGOALS)
 
-clean::
-  $(MAKE) -f $(SRCDIR)/$(LIB_PREFIX)_a.mk clean
+svr4lib_b_quote clean::
+  $(MAKE) -f $(MAKEFILE_RULES) \
+    DYLIB_NAME=svr4lib_b\\\" DYLIB_CXX_SOURCES=svr4lib_b_quote.cpp \
+    DYLIB_ONLY=YES $(MAKECMDGOALS)

lldb/packages/Python/lldbsuite/test/tools/lldb-server/libraries-svr4/svr4lib_a.mk

-LIB_PREFIX := svr4lib
-
-DYLIB_NAME := $(LIB_PREFIX)_a
-DYLIB_CXX_SOURCES := $(LIB_PREFIX)_a.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules

lldb/packages/Python/lldbsuite/test/tools/lldb-server/libraries-svr4/svr4lib_b_quote.mk

-LIB_PREFIX := svr4lib
-
-DYLIB_NAME := $(LIB_PREFIX)_b\"
-DYLIB_CXX_SOURCES := $(LIB_PREFIX)_b_quote.cpp
-DYLIB_ONLY := YES
-
-include Makefile.rules