[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 04:21:05 PDT 2020


sstefan1 marked an inline comment as done.
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='+')
----------------
lebedev.ri wrote:
> sstefan1 wrote:
> > 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?
> Why would that test require attributor?
> Just check that some attribute isn't dropped at `-O0`?
Ok, makes sense. Will do 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