[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