[PATCH] D81242: [StackSafety] Run ThinLTO

Fri Jul 31 10:58:25 PDT 2020

tejohnson added a comment.

In D81242#2183383 <https://reviews.llvm.org/D81242#2183383>, @tejohnson wrote:

> I just noticed that generateParamAccessSummary is taking a bit over 5% of the ThinLTO thin link step in an internal build (and will soon be more than 5% as I found another analysis that is hogging compile time that I'm going to work on fixing).

FYI with D84985 <https://reviews.llvm.org/D84985> now committed, this goes up to >7% of the runtime.

> This is currently the second hottest analysis in the thin link. Is the stack safety analysis meant to be always on with ThinLTO?
>
> I have a theory on what is causing it to incur so much overhead, see below.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81242