[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