[PATCH] D52095: Introduce explicit add_file_reading_unittest target for tests that use llvm::getInputFileDirectory()

[Zachary Turner via Phabricator via llvm-commits]

zturner accepted this revision.
zturner added inline comments.
This revision is now accepted and ready to land.


================
Comment at: unittests/DebugInfo/PDB/CMakeLists.txt:7
 
-add_llvm_unittest(DebugInfoPDBTests
+add_llvm_file_reading_unittest(DebugInfoPDBTests
   HashTableTest.cpp
----------------
Can you call this `add_llvm_unittest_with_input_files`?  The distinction is subtle, but we already have plenty of unittests that read files and don't need this functionality (specifically, most of the tests of `Support/FileSystem.h` itself.  This way it's clear what's being read.


https://reviews.llvm.org/D52095