idlecode added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/list.rst:156 + safety-no-assembler + safety-no-vector-bool ---------------- `safety-no-vector-bool` seems to belong to the other patch. ================ Comment at: clang-tools-extra/test/clang-tidy/safety-no-assembler.cpp:2 +// RUN: %check_clang_tidy %s safety-no-assembler %t + +void f() { ---------------- Maybe this check should handle `FileScopeAsmDecl` and `AsmLabelAttr` too? ``` __asm__(".symver foo, bar@v"); static int s asm("spam"); ``` https://reviews.llvm.org/D29267 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits