[PATCH] D41259: [debuginfo] Remove temporary FIXME.

Don Hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 14 13:54:21 PST 2017

hintonda created this revision.
hintonda added reviewers: zturner, aprantl.
Herald added a subscriber: JDevlieghere.

Now that r320495, "[debuginfo-tests] Support moving
debuginfo-tests to llvm/projects," has landed, remove temporary FIXME
that supported the old mechanism.

  rC Clang



Index: test/lit.cfg.py
--- test/lit.cfg.py
+++ test/lit.cfg.py
@@ -31,7 +31,7 @@
 # excludes: A list of directories to exclude from the testsuite. The 'Inputs'
 # subdirectories contain auxiliary inputs for various tests in their parent
 # directories.
-config.excludes = ['Inputs', 'CMakeLists.txt', 'README.txt', 'LICENSE.txt']
+config.excludes = ['Inputs', 'CMakeLists.txt', 'README.txt', 'LICENSE.txt', 'debuginfo-tests']
 # test_source_root: The root path where tests are located.
 config.test_source_root = os.path.dirname(__file__)
@@ -62,16 +62,6 @@
         'clang-func-mapping'), unresolved='ignore'),
-# FIXME: This logic can be removed once all buildbots have moved
-# debuginfo-test from clang/test to llvm/projects or monorepo.
-if os.path.exists(os.path.join(config.test_source_root, 'debuginfo-tests')):
-  if os.path.isfile(
-      os.path.join(config.test_source_root, 'debuginfo-tests', 'lit.cfg.py')):
-    config.excludes.append('debuginfo-tests')
-  else:
-    tools.append(ToolSubst('%test_debuginfo', command=os.path.join(
-      config.llvm_src_root, 'utils', 'test_debuginfo.pl')))
 if config.clang_examples:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41259.127019.patch
Type: text/x-patch
Size: 1253 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171214/e04e9efb/attachment.bin>

More information about the cfe-commits mailing list