[PATCH] No need to force-create clang-tools-extra lit.site.cfg

Edwin Vane edwin.vane at intel.com
Thu Feb 28 13:04:09 PST 2013


Hi echristo, klimek,

The make (all) target takes care of creating lit configs and
auto-generating tests. The problem with the original lit.site.cfg target
is it's not recursive and doesn't fully create everything necessary for
testing clang-tools-extra.

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

Files:
  test/Makefile

Index: test/Makefile
===================================================================
--- test/Makefile
+++ test/Makefile
@@ -60,11 +60,6 @@
 
 ifeq ($(shell test -f $(PROJ_OBJ_DIR)/../tools/clang/tools/extra/Makefile && echo OK), OK)
 LIT_ALL_TESTSUITES += $(PROJ_OBJ_DIR)/../tools/clang/tools/extra/test
-
-# Force creation of Clang Tools' lit.site.cfg.
-clang-tools-site-cfg: FORCE
-	$(MAKE) -C $(PROJ_OBJ_DIR)/../tools/clang/tools/extra/test lit.site.cfg
-extra-site-cfgs:: clang-tools-site-cfg
 endif
 endif
 endif
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D480.1.patch
Type: text/x-patch
Size: 523 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130228/030e1e40/attachment.bin>


More information about the cfe-commits mailing list