[cfe-commits] [PATCH] When using "make VERBOSE=1", make sure compiler-rt is also build verbosely.

Jean-Daniel Dupas devlists at shadowlab.org
Thu Oct 18 05:41:09 PDT 2012


When building clang+compiler-rt verbosely, the build commands of compiler-rt are not output in the terminal as expected.
This patch fixes the issue by forwarding the VERBOSE flag to the compiler-rt make.

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

Files:
  runtime/compiler-rt/Makefile

Index: runtime/compiler-rt/Makefile
===================================================================
--- runtime/compiler-rt/Makefile
+++ runtime/compiler-rt/Makefile
@@ -121,6 +121,12 @@
 # modified based on changes in the compiler-rt layout or build system.
 ####
 
+ifdef VERBOSE
+	RUNTIME_VERBOSE := VERBOSE=$(VERBOSE)
+else
+	RUNTIME_VERBOSE :=
+endif
+
 # Rule to build the compiler-rt libraries we need.
 #
 # We build all the libraries in a single shot to avoid recursive make as much as
@@ -130,12 +136,14 @@
 	  ProjSrcRoot=$(COMPILERRT_SRC_ROOT) \
 	  ProjObjRoot=$(PROJ_OBJ_DIR) \
 	  CC="$(ToolDir)/clang" \
+	  $(RUNTIME_VERBOSE) \
 	  $(RuntimeDirs:%=clang_%)
 .PHONY: BuildRuntimeLibraries
 CleanRuntimeLibraries:
 	$(Verb) $(MAKE) -C $(COMPILERRT_SRC_ROOT) \
 	  ProjSrcRoot=$(COMPILERRT_SRC_ROOT) \
 	  ProjObjRoot=$(PROJ_OBJ_DIR) \
+	  $(RUNTIME_VERBOSE) \
 	  clean
 .PHONY: CleanRuntimeLibraries
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67.1.patch
Type: text/x-patch
Size: 922 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121018/b6567fe2/attachment.bin>


More information about the cfe-commits mailing list