[PATCH] D83350: [test] Run llvm/test/**/*.yaml

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 7 15:20:15 PDT 2020


MaskRay created this revision.
MaskRay added reviewers: grimar, jhenderson, rupprecht.
Herald added subscribers: llvm-commits, cmtice.
Herald added a project: LLVM.

This patch extends D58439 <https://reviews.llvm.org/D58439> (`llvm/test/{yaml2obj,obj2yaml}/**/*.yaml`) and runs all
`llvm/test/**/*.yaml`

Many directories have configured `.yaml` (see the deleted lit.local.cfg
files). Yet still some don't configure .yaml and have caused stale tests:

- 8c5825befb7bbb2e76f7eccedc6d3bf26e9b2a6a <https://reviews.llvm.org/rG8c5825befb7bbb2e76f7eccedc6d3bf26e9b2a6a> test/llvm-readobj
- bdc3134e237737dd46b51cd1ecd41ecbbe9f921a <https://reviews.llvm.org/rGbdc3134e237737dd46b51cd1ecd41ecbbe9f921a> test/ExecutionEngine

Just hoist .yaml to `llvm/test/`. The number of tests running on my
machine increases from 38304 to 38309.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83350

Files:
  llvm/test/Object/lit.local.cfg
  llvm/test/ObjectYAML/lit.local.cfg
  llvm/test/lit.cfg.py
  llvm/test/tools/llvm-as/lit.local.cfg
  llvm/test/tools/llvm-dwarfdump/lit.local.cfg
  llvm/test/tools/llvm-gsymutil/ARM_AArch64/lit.local.cfg
  llvm/test/tools/llvm-gsymutil/X86/lit.local.cfg
  llvm/test/tools/llvm-nm/lit.local.cfg
  llvm/test/tools/llvm-objdump/lit.local.cfg
  llvm/test/tools/llvm-readobj/COFF/lit.local.cfg
  llvm/test/tools/llvm-xray/X86/lit.local.cfg
  llvm/test/tools/obj2yaml/lit.local.cfg
  llvm/test/tools/yaml2obj/lit.local.cfg


Index: llvm/test/tools/yaml2obj/lit.local.cfg
===================================================================
--- llvm/test/tools/yaml2obj/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.test', '.yaml']
Index: llvm/test/tools/obj2yaml/lit.local.cfg
===================================================================
--- llvm/test/tools/obj2yaml/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.test', '.yaml']
Index: llvm/test/tools/llvm-xray/X86/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-xray/X86/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.yaml', '.ll', '.txt']
Index: llvm/test/tools/llvm-readobj/COFF/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-readobj/COFF/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes.add('.yaml')
Index: llvm/test/tools/llvm-objdump/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-objdump/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.test', '.ll', '.s', '.yaml', '.txt']
Index: llvm/test/tools/llvm-nm/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-nm/lit.local.cfg
+++ llvm/test/tools/llvm-nm/lit.local.cfg
@@ -1,4 +1,2 @@
 if not 'X86' in config.root.targets:
     config.unsupported = True
-
-config.suffixes = ['.ll', '.s', '.test', '.yaml']
Index: llvm/test/tools/llvm-gsymutil/X86/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-gsymutil/X86/lit.local.cfg
+++ llvm/test/tools/llvm-gsymutil/X86/lit.local.cfg
@@ -1,4 +1,2 @@
 if not 'X86' in config.root.targets:
     config.unsupported = True
-
-config.suffixes = ['.test', '.yaml']
Index: llvm/test/tools/llvm-gsymutil/ARM_AArch64/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-gsymutil/ARM_AArch64/lit.local.cfg
+++ llvm/test/tools/llvm-gsymutil/ARM_AArch64/lit.local.cfg
@@ -1,4 +1,2 @@
 if not ('ARM' in config.root.targets and 'AArch64' in config.root.targets):
     config.unsupported = True
-
-config.suffixes = ['.test', '.yaml']
Index: llvm/test/tools/llvm-dwarfdump/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-dwarfdump/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.test', '.ll', '.s', '.yaml']
Index: llvm/test/tools/llvm-as/lit.local.cfg
===================================================================
--- llvm/test/tools/llvm-as/lit.local.cfg
+++ llvm/test/tools/llvm-as/lit.local.cfg
@@ -1,4 +1,2 @@
 if not 'X86' in config.root.targets:
     config.unsupported = True
-
-config.suffixes = ['.ll', '.s', '.test', '.yaml']
Index: llvm/test/lit.cfg.py
===================================================================
--- llvm/test/lit.cfg.py
+++ llvm/test/lit.cfg.py
@@ -22,7 +22,7 @@
 
 # suffixes: A list of file extensions to treat as test files. This is overriden
 # by individual lit.local.cfg files in the test subdirectories.
-config.suffixes = ['.ll', '.c', '.cxx', '.test', '.txt', '.s', '.mir']
+config.suffixes = ['.ll', '.c', '.cxx', '.test', '.txt', '.s', '.mir', '.yaml']
 
 # excludes: A list of directories to exclude from the testsuite. The 'Inputs'
 # subdirectories contain auxiliary inputs for various tests in their parent
Index: llvm/test/ObjectYAML/lit.local.cfg
===================================================================
--- llvm/test/ObjectYAML/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.yaml']
Index: llvm/test/Object/lit.local.cfg
===================================================================
--- llvm/test/Object/lit.local.cfg
+++ /dev/null
@@ -1 +0,0 @@
-config.suffixes = ['.test', '.ll', '.s', '.yaml']


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83350.276231.patch
Type: text/x-patch
Size: 3904 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200707/2e611c69/attachment.bin>


More information about the llvm-commits mailing list