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