[PATCH] D17981: [clang-tidy] Fix clang-tidy to support parsing of assembly statements.
Reid Kleckner via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 10 13:15:14 PST 2016
rnk added a comment.
This will definitely work, but it will substantially increase the size of clang-tidy:
$ du -cksh bin/clang-tidy.exe bin/clang.exe
21M bin/clang-tidy.exe
47M bin/clang.exe
68M total
The difference is mostly from the backends, and that's with only the X86 and ARM backends enabled.
Alex, how should clang-tidy behave when it sees MS inline asm? If we ignore the error about not finding a registered target, then we treat the asm statement as empty. This may cause false positives where clang-tidy doesn't see Decl uses from inline asm, but that may be acceptable, since they mostly come from system headers.
http://reviews.llvm.org/D17981
More information about the cfe-commits
mailing list