[PATCH] D73708: [update_cc_test_checks] Don't attach CHECK lines to function declarations

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 3 18:14:15 PST 2020


greened added a comment.

LGTM.  I did something similar in my local copy that I was going to upstream but I added a switch for the user to choose where to place checks (declaration, definition or both).  I only added the switch because I wasn't sure whether the changed behavior was intended.  I'm fine just placing checks on definitions as I don't have a need to put them on declarations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73708





More information about the llvm-commits mailing list