[PATCH] D120993: [PassManager] Add pretty stack entries before P->run() call.

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 9 09:18:21 PST 2022


aeubanks added a comment.

In D120993#3370129 <https://reviews.llvm.org/D120993#3370129>, @fhahn wrote:

> In D120993#3370083 <https://reviews.llvm.org/D120993#3370083>, @nikic wrote:
>
>> FYI it looks like this isn't as free as I would have expected: http://llvm-compile-time-tracker.com/compare.php?from=338dfcd60f843082bb589b287d890dbd9394eb82&to=128745cc2681c284bc6d0150a319673a6d6e8424&stat=instructions
>
> Yeah that's not great. I'll revert it in a bit and check if there's anything in particular that's very expensive.

If I had to guess, it might be eagerly retrieving loop names or cgscc names since they can be pretty big if the scc contains a lot of functions or if the loop has a lot of blocks
we could start without the cgscc name and using the function name for loop passes


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120993



More information about the llvm-commits mailing list