[PATCH] D68161: [TimeProfiler] Fix "OptModule" section and add new "Backend" sections
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 27 23:55:07 PDT 2019
anton-afanasyev added a comment.
In D68161#1686632 <https://reviews.llvm.org/D68161#1686632>, @lebedev.ri wrote:
> Oh nice!
> F10096481: image.png <https://reviews.llvm.org/F10096481>
> F10096488: image.png <https://reviews.llvm.org/F10096488>
> I like this.
>
> One more related question - there are two `"Frontend"` sections, second one appears to consist only of `"CodeGen Function"s`.
> Is that correct? Should they both be in an actual front-end section, and should they be renamed to, i dunno,
> `"Sema"` and `"Codegen"`? I'm not sure about this, just raising a question.
Ok, committing this.
As for two `"Frontend"` sections -- we've discussed this several months ago (https://reviews.llvm.org/D63325#1544555), it is not obvious how to join these sections to one (start and finish of such section are separated a lot and the code for joining is opaque).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68161/new/
https://reviews.llvm.org/D68161
More information about the llvm-commits
mailing list