Add missing dependencies on the flang test target

joker-eph · joker-eph · commit ed6c452edc81 · 2020-04-13T18:23:01.000Z
Without this, the LLVM utilities (FileCheck) aren't built when running `ninja check-flang` and it fails with: llvm-project/llvm/utils/lit/lit/llvm/subst.py:134: fatal: Did not find FileCheck in... Also the modules aren't built without depending on `module_files`, which makes multiple tests failing. Differential Revision: https://reviews.llvm.org/D78036
diff --git a/flang/test/CMakeLists.txt b/flang/test/CMakeLists.txt
@@ -18,7 +18,7 @@ set(FLANG_TEST_PARAMS
   flang_site_config=${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg.py)
 
 set(FLANG_TEST_DEPENDS
-  f18
+  f18 FileCheck count not module_files
 )
 
 if (LINK_WITH_FIR)

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ set(FLANG_TEST_PARAMS`
`18`	`18`	`flang_site_config=${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg.py)`
`19`	`19`
`20`	`20`	`set(FLANG_TEST_DEPENDS`
`21`		`- f18`
	`21`	`+ f18 FileCheck count not module_files`
`22`	`22`	`)`
`23`	`23`
`24`	`24`	`if (LINK_WITH_FIR)`