[PATCH] D83004: [UpdateCCTestChecks] Include generated functions if asked

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 15:41:12 PDT 2020


jdoerfert added a comment.

This is great! Just a few days ago I added a TODO in one of the tests I created asking for this: D82722 <https://reviews.llvm.org/D82722> :)

Will this work for all test scripts?

Will this make the `prefix_exclusions` logic obsolete?



================
Comment at: llvm/utils/update_cc_test_checks.py:133
+  parser.add_argument('--include-generated-funcs', action='store_true',
+                      help='Output checks for functions not in source')
   parser.add_argument('tests', nargs='+')
----------------
I think this should go into common.py (after D78618). I would also make this the default but OK.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83004





More information about the llvm-commits mailing list