<div dir="ltr">LGTM.<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 28, 2013 at 12:58 PM, Edwin Vane <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi gribozavr, echristo,<br>
<br>
Autoconf make (all) now properly recurses from tools/extra/Makefile into<br>
tools/extra/test/Makefile and tools/extra/test/cpp11-migrate/Makefile.<br>
The 'all' target is responsible for creating lit config files and<br>
autogenerating tests. Subsequent 'check-all' targets will properly work.<br>
<br>
Re-enabling UseAuto/iterator.cpp test.<br>
<br>
General clean-up of clang-tools-extra makefiles, removing dead targets<br>
and not using copied pieces of Makefile.rules.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D479" target="_blank">http://llvm-reviews.chandlerc.com/D479</a><br>
<br>
Files:<br>
  Makefile<br>
  test/Makefile<br>
  test/cpp11-migrate/Makefile<br>
  test/cpp11-migrate/UseAuto/iterator.cpp<br>
<br>
Index: Makefile<br>
===================================================================<br>
--- Makefile<br>
+++ Makefile<br>
@@ -12,29 +12,15 @@<br>
 include $(CLANG_LEVEL)/../../Makefile.config<br>
<br>
 PARALLEL_DIRS := remove-cstr-calls tool-template clang-format cpp11-migrate<br>
+DIRS := test<br>
<br>
 include $(CLANG_LEVEL)/Makefile<br>
<br>
-###<br>
-# Handle the nested test suite.<br>
-<br>
-ifneq ($(PROJ_SRC_ROOT),$(PROJ_OBJ_ROOT))<br>
-$(RecursiveTargets)::<br>
-       $(Verb) for dir in test; do \<br>
-         if [ -f $(PROJ_SRC_DIR)/$${dir}/Makefile ] && [ ! -f $${dir}/Makefile ]; then \<br>
-           $(MKDIR) $${dir}; \<br>
-           $(CP) $(PROJ_SRC_DIR)/$${dir}/Makefile $${dir}/Makefile; \<br>
-         fi \<br>
-       done<br>
-endif<br>
-<br>
+# Custom target. Pass request to test/Makefile that knows what to do. To access<br>
+# this target you'd issue:<br>
+#<br>
+# make -C <build_dir>/tools/clang/tools/extra test<br>
 test::<br>
-       @ $(MAKE) -C test<br>
-<br>
-report::<br>
-       @ $(MAKE) -C test report<br>
-<br>
-clean::<br>
-       @ $(MAKE) -C test clean<br>
+       @ $(MAKE) -C test test<br>
<br>
-.PHONY: test report clean<br>
+.PHONY: test<br>
Index: test/Makefile<br>
===================================================================<br>
--- test/Makefile<br>
+++ test/Makefile<br>
@@ -9,8 +9,6 @@<br>
<br>
 CLANG_LEVEL := ../../..<br>
 include $(CLANG_LEVEL)/../../Makefile.config<br>
-<br>
-# Recurse into any subdirectories that have their own Makefiles.<br>
 DIRS := cpp11-migrate<br>
 include $(CLANG_LEVEL)/Makefile<br>
<br>
@@ -45,14 +43,19 @@<br>
 endif<br>
<br>
 all:: lit.site.cfg<br>
+<br>
+# Run just the Clang extra tools tests. This target assumes 'make (all)' has<br>
+# been run previously as that target is responsible for generating lit config<br>
+# files and auto-generated tests.<br>
+test::<br>
        @ echo '--- Running the Clang extra tools tests for $(TARGET_TRIPLE) ---'<br>
        @ $(PYTHON) $(LLVM_SRC_ROOT)/utils/lit/lit.py \<br>
          $(LIT_ARGS) $(TESTARGS) $(TESTDIRS)<br>
<br>
 FORCE:<br>
<br>
 lit.site.cfg: FORCE<br>
-       @echo "Making Clang extra tools' 'lit.site.cfg' file..."<br>
+       @echo "Making lit.site.cfg for Clang extra tools..."<br>
        @$(ECHOPATH) s=@LLVM_SOURCE_DIR@=$(LLVM_SRC_ROOT)=g > lit.tmp<br>
        @$(ECHOPATH) s=@LLVM_BINARY_DIR@=$(LLVM_OBJ_ROOT)=g >> lit.tmp<br>
        @$(ECHOPATH) s=@LLVM_TOOLS_DIR@=$(ToolDir)=g >> lit.tmp<br>
@@ -69,4 +72,4 @@<br>
 clean::<br>
        @ find . -name Output | xargs rm -fr<br>
<br>
-.PHONY: all report clean<br>
+.PHONY: all test clean<br>
Index: test/cpp11-migrate/Makefile<br>
===================================================================<br>
--- test/cpp11-migrate/Makefile<br>
+++ test/cpp11-migrate/Makefile<br>
@@ -22,7 +22,6 @@<br>
   UseAuto/<a href="http://gen_basic_std_iterator_tests.cpp.py" target="_blank">gen_basic_std_iterator_tests.cpp.py</a> \<br>
   UseAuto/Inputs/<a href="http://gen_my_std.h.py" target="_blank">gen_my_std.h.py</a><br>
<br>
-<br>
 # macro to be used with $(call) that generates a rule and recipe that causes a<br>
 # file to be auto-generated from a generator script. Generator scripts must<br>
 # follow scheme above. The resulting file is placed in:<br>
Index: test/cpp11-migrate/UseAuto/iterator.cpp<br>
===================================================================<br>
--- test/cpp11-migrate/UseAuto/iterator.cpp<br>
+++ test/cpp11-migrate/UseAuto/iterator.cpp<br>
@@ -1,6 +1,6 @@<br>
 // RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
-// NORUN cpp11-migrate -use-auto %t.cpp -- --std=c++11 -I %gen_root/UseAuto/Inputs<br>
-// NORUN FileCheck -input-file=%t.cpp %s<br>
+// RUN: cpp11-migrate -use-auto %t.cpp -- --std=c++11 -I %gen_root/UseAuto/Inputs<br>
+// RUN: FileCheck -input-file=%t.cpp %s<br>
 #include "my_std.h"<br>
<br>
 typedef std::vector<int>::iterator int_iterator;<br>
</blockquote></div><br></div>