[PATCH] D83629: [Utils] Check function attributes in update_test_checks

Stefan Stipanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 19 03:48:48 PDT 2020


sstefan1 marked an inline comment as done.
sstefan1 added a subscriber: lebedev.ri.
sstefan1 added inline comments.


================
Comment at: llvm/utils/update_test_checks.py:56
+  parser.add_argument('--check-attributes', action='store_true',
+                      help='Check "Function Attributes" for functions')
   parser.add_argument('tests', nargs='+')
----------------
jdoerfert wrote:
> Can we add this to the _cc_test script too (with test). Seems useful when you come from C as well.
Sure. One concern though. I think this would require running the test with at least `-O1` and `-attributor-enable` flags. 
I remember @lebedev.ri  having concerns about this. Should I go ahead with that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83629





More information about the llvm-commits mailing list