[PATCH] D158239: [clang][ExtractAPI] Add support for namespaces

Erick Velez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 21 15:31:24 PDT 2023


evelez7 updated this revision to Diff 552155.
evelez7 marked an inline comment as done.
evelez7 added a comment.

Move DetermineParentDecl to determineParentRecord in APIVisitor as a private method.

Easier to just let it fetch the record for us and return nullptr if the parent is the translation unit.
Also we don't have to cast the Decls to their most derived form for the USR.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158239

Files:
  clang/include/clang/ExtractAPI/API.h
  clang/include/clang/ExtractAPI/DeclarationFragments.h
  clang/include/clang/ExtractAPI/ExtractAPIVisitor.h
  clang/include/clang/ExtractAPI/Serialization/SerializerBase.h
  clang/include/clang/ExtractAPI/Serialization/SymbolGraphSerializer.h
  clang/lib/ExtractAPI/API.cpp
  clang/lib/ExtractAPI/DeclarationFragments.cpp
  clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp
  clang/test/ExtractAPI/namespace.cpp
  clang/test/ExtractAPI/nested_namespaces.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D158239.552155.patch
Type: text/x-patch
Size: 34811 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230821/cf90c667/attachment-0001.bin>


More information about the cfe-commits mailing list