[PATCH] D123672: [clangd] Export preamble AST and serialized size as metrics

2022-04-13 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGf407c9ed1057: [clangd] Export preamble AST and serialized size as metrics (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123672/new/

[PATCH] D123672: [clangd] Export preamble AST and serialized size as metrics

2022-04-13 Thread Sam McCall via Phabricator via cfe-commits
sammccall added inline comments. Comment at: clang-tools-extra/clangd/Preamble.cpp:500 + if (Stats != nullptr) { +Stats->TotalBuildTime = PreambleTimer.getTime(); adamcz wrote: > This is a significant change. You are now exporting this information for > f

[PATCH] D123672: [clangd] Export preamble AST and serialized size as metrics

2022-04-13 Thread Adam Czachorowski via Phabricator via cfe-commits
adamcz added inline comments. Comment at: clang-tools-extra/clangd/Preamble.cpp:500 + if (Stats != nullptr) { +Stats->TotalBuildTime = PreambleTimer.getTime(); This is a significant change. You are now exporting this information for failed preamble builds

[PATCH] D123672: [clangd] Export preamble AST and serialized size as metrics

2022-04-13 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: adamcz. Herald added subscribers: usaxena95, kadircet, arphaman, javed.absar. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: