[PATCH] D68161: [TimeProfiler] Fix "OptModule" section and add new "Backend" sections

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