[Lldb-commits] [lldb] [lldb] replace usage of $(RM) in Makefile (PR #180755)
Charles Zablit via lldb-commits
lldb-commits at lists.llvm.org
Tue Feb 10 07:33:13 PST 2026
https://github.com/charles-zablit created https://github.com/llvm/llvm-project/pull/180755
This patch replaces the usages of `$(RM)` with cross platform `$(call RM,...)` calls which was added in https://github.com/llvm/llvm-project/pull/180224.
>From 97d19d8a05ce54b5c5b215ae88acb908d224c358 Mon Sep 17 00:00:00 2001
From: Charles Zablit <c_zablit at apple.com>
Date: Tue, 10 Feb 2026 16:31:14 +0100
Subject: [PATCH] [lldb] replace usage of $(RM) in Makefile
---
lldb/packages/Python/lldbsuite/test/make/Makefile.rules | 2 +-
lldb/test/API/functionalities/disassembler-variables/Makefile | 2 +-
lldb/test/API/functionalities/module_cache/bsd/Makefile | 4 ++--
lldb/test/API/linux/sepdebugsymlink/Makefile | 4 ++--
4 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/lldb/packages/Python/lldbsuite/test/make/Makefile.rules b/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
index 4109670ebe64c..eeaf651792fee 100644
--- a/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
+++ b/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
@@ -730,7 +730,7 @@ clean::
ifeq "$(findstring lldb-test-build.noindex, $(BUILDDIR))" ""
$(error Trying to invoke the clean rule, but not using the default build tree layout)
else
- $(RM) -r $(wildcard $(BUILDDIR)/*)
+ $(call RM_RF,$(wildcard $(BUILDDIR)/*))
endif
#----------------------------------------------------------------------
diff --git a/lldb/test/API/functionalities/disassembler-variables/Makefile b/lldb/test/API/functionalities/disassembler-variables/Makefile
index 0b6a8cc6bb297..2be51d8d3d774 100644
--- a/lldb/test/API/functionalities/disassembler-variables/Makefile
+++ b/lldb/test/API/functionalities/disassembler-variables/Makefile
@@ -27,6 +27,6 @@ all: $(ASM_OBJS) $(EXE)
# Keeping things tidy.
clean::
- $(RM) -f $(ASM_OBJS) dummy_main.c
+ $(call RM_RF,$(ASM_OBJS) dummy_main.c)
include Makefile.rules
diff --git a/lldb/test/API/functionalities/module_cache/bsd/Makefile b/lldb/test/API/functionalities/module_cache/bsd/Makefile
index 8060c20f7d5be..751eec8eb799e 100644
--- a/lldb/test/API/functionalities/module_cache/bsd/Makefile
+++ b/lldb/test/API/functionalities/module_cache/bsd/Makefile
@@ -9,7 +9,7 @@ a.out: main.o libfoo.a
lib_ab.a: a.o b.o
$(AR) $(ARFLAGS) $@ $^
- $(RM) $^
+ $(call RM,$^)
# Here we make a .a file that has two a.o files with different modification
# times and different content by first creating libfoo.a with only a.o and b.o,
@@ -22,6 +22,6 @@ libfoo.a: lib_ab.a c.o
touch c.o
mv c.o a.o
$(AR) $(ARFLAGS) $@ lib_ab.a a.o
- $(RM) a.o
+ $(call RM,a.o)
include Makefile.rules
diff --git a/lldb/test/API/linux/sepdebugsymlink/Makefile b/lldb/test/API/linux/sepdebugsymlink/Makefile
index a1b84c1843e96..ad6a17c0d7530 100644
--- a/lldb/test/API/linux/sepdebugsymlink/Makefile
+++ b/lldb/test/API/linux/sepdebugsymlink/Makefile
@@ -3,13 +3,13 @@ C_SOURCES := main.c
all: dirsymlink
dirreal: a.out
- $(RM) -r $@
+ $(call RM_RF,$@)
mkdir $@
$(OBJCOPY) --only-keep-debug $< $@/stripped.debug
$(OBJCOPY) --strip-all --add-gnu-debuglink=$@/stripped.debug $< $@/stripped.out
dirsymlink: dirreal
- $(RM) -r $@
+ $(call RM_RF,$@)
mkdir $@
ln -s ../$</stripped.out $@/stripped.symlink
More information about the lldb-commits
mailing list