[PATCH] D78618: [UpdateTestChecks] Move more update_test_checks.py logic to common.py

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 09:42:31 PDT 2020


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

Nice cleanup! Sorry for the delay. LGTM



================
Comment at: llvm/utils/UpdateTestChecks/common.py:101
+      yield TestInfo(test, parser, script_name, input_lines, args, argv,
+                     comment_prefix)
+
----------------
Nit: Should we look for `UTC_ADVERT` instead of the `'autogenerated'` string?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78618





More information about the llvm-commits mailing list