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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to