This is an archive of the discontinued LLVM Phabricator instance.

Rely on Makefile.rules to clean up the lldb\test\driver\batch_mode directory.
ClosedPublic

Authored by amccarth on Mar 16 2015, 1:18 PM.

Download Raw Diff

Details

Reviewers

Commits

rG5028e33e5468: Rely on Makefile.rules in lldb/test/driver/batch_mode.
rLLDB232974: Rely on Makefile.rules in lldb/test/driver/batch_mode.
rL232974: Rely on Makefile.rules in lldb/test/driver/batch_mode.

Summary

As far as I can tell, this custom makefile doesn't do anything special. Relying on the Makefile.rules like the other tests allows this directory to be cleaned properly after running the test.

Diff Detail

Event Timeline

amccarth updated this revision to Diff 22049.Mar 16 2015, 1:18 PM

amccarth retitled this revision from to Rely on Makefile.rules to clean up the lldb\test\driver\batch_mode directory..

amccarth updated this object.

amccarth edited the test plan for this revision. (Show Details)

amccarth added a reviewer: zturner.

amccarth added a subscriber: Unknown Object (MLST).

Bump. Did you already submit this one Zach?

lgtm

This revision is now accepted and ready to land.Mar 23 2015, 11:02 AM

Closed by commit rL232974: Rely on Makefile.rules in lldb/test/driver/batch_mode. (authored by zturner). · Explain WhyMar 23 2015, 11:02 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents

Path

Size

test/

driver/

batch_mode/

Makefile

29 lines

Diff 22049

test/driver/batch_mode/Makefile

	CC ?= clang			LEVEL = ../../make
	ifeq "$(ARCH)" ""
	ARCH = x86_64
	endif

	ifeq "$(OS)" ""			C_SOURCES := main.c
	OS = $(shell uname -s)
	endif

	CFLAGS ?= -g -O0			include $(LEVEL)/Makefile.rules
	CWD := $(shell pwd)

	LIB_PREFIX := lib

	ifeq "$(OS)" "Darwin"
	CFLAGS += -arch $(ARCH)
	endif

	all: a.out

	a.out: main.o
	$(CC) $(CFLAGS) -o a.out main.o

	main.o: main.c
	$(CC) $(CFLAGS) -c main.c

	clean:
	rm -rf $(wildcard .o ~ .dylib .so a.out *.dSYM)