[PATCH] D112024: [clang] diagnose_as attribute for Fortify diagnosing like builtins.

Michael Benfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 23 16:45:38 PST 2021


mbenfield updated this revision to Diff 389355.
mbenfield added a comment.

Error on member function. Test case for this.

Test case for attribute applied to a non-function declaration.

Handle variadic functions. Document this. Test case for variadic functions.

Remove const from some local values.

Spell out some types.

Change superflous `dyn_cast_or_null` to `dyn_cast`.

Remove some curly braces.

Pass in a `NamedDecl` directly rather than its name to a diagnostic.

Add a test case for types not being identical.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112024/new/

https://reviews.llvm.org/D112024

Files:
  clang/include/clang/Basic/Attr.td
  clang/include/clang/Basic/AttrDocs.td
  clang/include/clang/Basic/DiagnosticSemaKinds.td
  clang/include/clang/Sema/ParsedAttr.h
  clang/lib/AST/Decl.cpp
  clang/lib/Sema/SemaChecking.cpp
  clang/lib/Sema/SemaDeclAttr.cpp
  clang/test/Sema/attr-diagnose-as-builtin.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112024.389355.patch
Type: text/x-patch
Size: 18390 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20211124/2d3f7955/attachment-0001.bin>


More information about the cfe-commits mailing list