[libcxx-commits] [PATCH] D154987: [lit] Implement PYTHON directive and config.prologue

[Joel E. Denny via Phabricator via libcxx-commits]

jdenny added a comment.

In D154987#4491993 <https://reviews.llvm.org/D154987#4491993>, @arichardson wrote:

> Are you also planning to adjust the update_tests.py scripts to support this syntax?

Thanks, I hadn't thought of that.  Is that something you would like to use?  Regardless of use cases, I'm inclined to wait to see the reaction to PYTHON directives generally.

> Might be a good motivation to have them use the lit parser to extract commands rather than regexes.

Sounds reasonable to me, but I haven't spent much time with those scripts in a while.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154987