[PATCH] D151293: [clang][ExtractAPI] Refactor serializer to the CRTP

2023-05-25 Thread Erick Velez via Phabricator via cfe-commits
evelez7 abandoned this revision. evelez7 added inline comments. Comment at: clang/include/clang/ExtractAPI/Serialization/SerializerBase.h:28-31 +struct APISetVisitorOption { /// Do not include unnecessary whitespaces to save space. bool Compact; }; dang

[PATCH] D151293: [clang][ExtractAPI] Refactor serializer to the CRTP

2023-05-25 Thread Erick Velez via Phabricator via cfe-commits
evelez7 updated this revision to Diff 525652. evelez7 added a comment. Really address review feedback Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151293/new/ https://reviews.llvm.org/D151293 Files:

[PATCH] D151293: [clang][ExtractAPI] Refactor serializer to the CRTP

2023-05-24 Thread Erick Velez via Phabricator via cfe-commits
evelez7 updated this revision to Diff 525431. evelez7 marked 8 inline comments as done. evelez7 added a comment. Address some review feedback Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151293/new/ https://reviews.llvm.org/D151293 Files:

[PATCH] D151293: [clang][ExtractAPI] Refactor serializer to the CRTP

2023-05-24 Thread Daniel Grumberg via Phabricator via cfe-commits
dang requested changes to this revision. dang added a comment. This revision now requires changes to proceed. Great start but there are still some rough edges to polish! Comment at: clang/include/clang/ExtractAPI/Serialization/SerializerBase.h:19 #include

[PATCH] D151293: [clang][ExtractAPI] Refactor serializer to the CRTP

2023-05-23 Thread Erick Velez via Phabricator via cfe-commits
evelez7 created this revision. Herald added a reviewer: ributzka. Herald added a project: All. evelez7 requested review of this revision. Herald added a reviewer: dang. Herald added a project: clang. Herald added a subscriber: cfe-commits. Refactor SerializerBase and SymbolGraphSerializer to use