[PATCH] D80584: [utils] change update_test_checks.py use of 'TMP' value names

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 31 16:13:53 PDT 2020


jdoerfert reopened this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

This needs a command line flag so people can change the prefix. The default for that flag should be "TMP" to minimize the noise in updates. We also should have a test.
FWIW, the UTC_FLAGS logic will "remember" the flag chosen by the user so if you need to change from "TMP" to something else it will be recorded and no further interaction will be necessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80584





More information about the llvm-commits mailing list