rnk added a comment.

In D58737#1412847 <https://reviews.llvm.org/D58737#1412847>, @xur wrote:

> lgtm.
>
> BTW, I'm in the process of committing D54175 
> <https://reviews.llvm.org/D54175>. After that patch, PGO instrumentation can 
> be called after the main inlining. I don't think it will conflict anything in 
> this patch


I see, I guess I'll update the comment to say "may run before the inliner".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58737/new/

https://reviews.llvm.org/D58737



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

Reply via email to