[PATCH] D79423: [analyzer][NFC] StdLibraryFunctionsChecker: Add empty Signatures

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 7 09:05:21 PDT 2020


xazax.hun added a comment.

In D79423#2025001 <https://reviews.llvm.org/D79423#2025001>, @martong wrote:

> 1. Some function types contain non-builtin types. E.g. `FILE*`. We cannot get this type as easily as we do with builtin types (we can get builtins simply from the ASTContext). In case of such a compound type, we should be digging up the type from the AST, and that can be done by doing a lookup.


I see the problem, but as far as I understand, this is not unique to libC or Posix. Any libraries that use non-primitive types (which is most of them) will suffer from the very same problem. So I anticipated that this type lookup from the ASTContext needs to be implemented regardless of the irrelevant signature functionality.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79423





More information about the cfe-commits mailing list