[PATCH] D154280: [LIT] Added an option to llvm-lit to emit the necessary test coverage data, divided per test case
Henrik G Olsson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 4 02:27:07 PDT 2023
hnrklssn added inline comments.
================
Comment at: llvm/utils/lit/tests/Inputs/per-test-coverage/lit.cfg:7
+config.test_format = lit.formats.ShTest()
+config.substitutions.append(("%LLVM_PROFILE_FILE", os.environ.get('LLVM_PROFILE_FILE')))
----------------
Instead of fetching the environment variable here (which seems like it's probably happening before the test is executed?) as part of the test framework, try running a script that reads the env var on its own
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154280/new/
https://reviews.llvm.org/D154280
More information about the llvm-commits
mailing list