[PATCH] D157007: [clang][ExtractAPI] Add support for C++ classes with fix

2023-08-04 Thread Daniel Grumberg via Phabricator via cfe-commits
dang accepted this revision. dang added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157007/new/ https://reviews.llvm.org/D157007 ___ cfe

[PATCH] D157007: [clang][ExtractAPI] Add support for C++ classes with fix

2023-08-03 Thread Erick Velez via Phabricator via cfe-commits
evelez7 added inline comments. Comment at: clang/lib/ExtractAPI/DeclarationFragments.cpp:612 + if (isa(Method)) { +Name = Method->getNameAsString(); +if (dyn_cast(Method)->isExplicit()) Minor change is here. Since `Name` is now an `std::string`, a cast t

[PATCH] D157007: [clang][ExtractAPI] Add support for C++ classes with fix

2023-08-03 Thread Erick Velez via Phabricator via cfe-commits
evelez7 created this revision. evelez7 added a reviewer: dang. Herald added a reviewer: ributzka. Herald added a subscriber: ChuanqiXu. Herald added a project: All. evelez7 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Reintroduce D153557