[PATCH] D63325: [Support][Time profiler] Make FE codegen blocks to be inside frontend blocks
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 18 04:10:43 PDT 2019
anton-afanasyev added inline comments.
================
Comment at: clang/lib/CodeGen/CodeGenAction.cpp:232
{
+ llvm::TimeTraceScope TimeScope("Frontend", StringRef(""));
PrettyStackTraceString CrashInfo("Per-file LLVM IR generation");
----------------
lebedev.ri wrote:
> This looks more like `Frontend Codegen` to me?
> (With front-end itself being the entirety of clang time up to this point including this step)
`Codegen` sections are not inside `HandleTranslationUnit()` function only. So I've changed it other way: started `Frontend` section inside `ParseAST()` and finished this section inside `HandleTranslationUnit()`. Thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63325/new/
https://reviews.llvm.org/D63325
More information about the llvm-commits
mailing list