[PATCH] D78845: [COFF] Add a fastpath for /INCLUDE: in .drective sections

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 28 11:17:29 PDT 2020


aganea added a comment.

In D78845#2008164 <https://reviews.llvm.org/D78845#2008164>, @rnk wrote:

> I think early in LLVM project history, developers did a lot of micro-optimization focusing on reducing heap allocations (see prevalence (and overuse!) of SmallVector), and a lot of that has gone by the wayside as generic containers proliferate in new code.


I think the main issue is that there's no easy way to "see" how the allocations scale across LLVM in general. How many of them, where they are done, how many per sec., diff. against a previous build, etc.
We could implement a system to "tag" each allocation and save it along with the callstack. It would then easy to save binary snapshots and inspect where the allocations go, do diff-ing, etc.
It just takes someone that has time to do it ;-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78845





More information about the llvm-commits mailing list