[PATCH] D108023: [flang][lit] Introduce `%python_b` in LIT configuration

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 13 02:23:55 PDT 2021


awarzynski created this revision.
Herald added a reviewer: sscalpone.
Herald added a reviewer: clementval.
awarzynski requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a project: LLVM.

In order to prevent Python from generating the __pycache__ directory, we
need to pass the `-B` flag [1]. To this end, a new LIT variable is
introduced, `%python_b`, that expands to `python -B`.

This is currently only required in Flang as it uses
flang/test/Semantics/common.py that's imported in tests that use
`%python`. In order to speed-up re-use, this Python module is
pre-compiled by-default and saved in `flang/test/Semantics/__pycache__`.

This was brought up in a post-commit review in [2]

[1] https://docs.python.org/3/using/cmdline.html#id1
[2] https://reviews.llvm.org/D107041


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D108023

Files:
  flang/test/Semantics/OpenACC/acc-symbols01.f90
  flang/test/Semantics/kinds01.f90
  flang/test/Semantics/kinds03.f90
  flang/test/Semantics/omp-do-schedule03.f90
  flang/test/Semantics/omp-do-schedule04.f90
  flang/test/Semantics/omp-do01-positivecase.f90
  flang/test/Semantics/omp-do04-positivecase.f90
  flang/test/Semantics/omp-do05-positivecase.f90
  flang/test/Semantics/omp-do06-positivecases.f90
  flang/test/Semantics/omp-do11.f90
  flang/test/Semantics/omp-do12.f90
  flang/test/Semantics/omp-do14.f90
  flang/test/Semantics/omp-do17.f90
  flang/test/Semantics/omp-reduction08.f90
  flang/test/Semantics/omp-reduction09.f90
  flang/test/Semantics/omp-symbol01.f90
  flang/test/Semantics/omp-symbol02.f90
  flang/test/Semantics/omp-symbol03.f90
  flang/test/Semantics/omp-symbol04.f90
  flang/test/Semantics/omp-symbol05.f90
  flang/test/Semantics/omp-symbol06.f90
  flang/test/Semantics/omp-symbol07.f90
  flang/test/Semantics/omp-symbol08.f90
  flang/test/Semantics/procinterface01.f90
  flang/test/Semantics/symbol01.f90
  flang/test/Semantics/symbol02.f90
  flang/test/Semantics/symbol03.f90
  flang/test/Semantics/symbol05.f90
  flang/test/Semantics/symbol06.f90
  flang/test/Semantics/symbol07.f90
  flang/test/Semantics/symbol08.f90
  flang/test/Semantics/symbol09.f90
  flang/test/Semantics/symbol10.f90
  flang/test/Semantics/symbol11.f90
  flang/test/Semantics/symbol12.f90
  flang/test/Semantics/symbol13.f90
  flang/test/Semantics/symbol14.f90
  flang/test/Semantics/symbol15.f90
  flang/test/Semantics/symbol16.f90
  flang/test/Semantics/symbol17.f90
  flang/test/Semantics/symbol18.f90
  flang/test/Semantics/symbol19.f90
  flang/test/lit.cfg.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D108023.366224.patch
Type: text/x-patch
Size: 17566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210813/fdad4372/attachment-0001.bin>


More information about the llvm-commits mailing list