[PATCH] D138224: WIP: AST{Reader,Writer} alternative

2023-04-20 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 abandoned this revision. jansvoboda11 added a comment. Using `CompilerInstance` as the module storage dramatically increases memory usage (peak ~30GB) of `clang-scan-deps` when scanning LLVM. (Memory mapping serialized files peaks at ~300MB.) Repository: rG LLVM Github Monorepo

[PATCH] D138224: WIP: AST{Reader,Writer} alternative

2022-11-21 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a comment. ("Simple" tends to be an over used/overly vague term and may become less correct over time - if there's some more descriptive term that explains the difference between the old and newer one it'd be awesome to use - but yeah, sometimes "Simple", "New", etc, is just the

[PATCH] D138224: WIP: AST{Reader,Writer} alternative

2022-11-17 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 476205. jansvoboda11 added a comment. Prevent module redefinition errors, fix memory leak Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138224/new/ https://reviews.llvm.org/D138224 Files: clang/include/

[PATCH] D138224: WIP: AST{Reader,Writer} alternative

2022-11-17 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. Herald added a subscriber: ributzka. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D138224 File