[PATCH] D29267: [clang-tidy] safety-no-assembler

[Paweł Żukowski via Phabricator via cfe-commits]

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 at v");
static int s asm("spam");
```


https://reviews.llvm.org/D29267