[Openmp-commits] [PATCH] D122645: [OpenMP][NFC] lit: Allow setting default environment variables for test

[Joachim Protze via Phabricator via Openmp-commits]

protze.joachim added a comment.

I think, the name of the variable is confusing. The tests triggered and influenced by your example command are not limited to libomp. Currently the `LIBOMP_` prefix is mainly used for cmake variables. The make target to run the tests is called `check-*` rather than `test-*`.

My suggestion for the variable name would be `CHECK_OPENMP_ENV`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122645/new/

https://reviews.llvm.org/D122645