[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