[PATCH] D35989: [SCEV][NFC] Introduces expression sizes estimation

Mon Jan 21 12:28:58 PST 2019

sanjoy added a comment.

In D35989#1364854 <https://reviews.llvm.org/D35989#1364854>, @mkazantsev wrote:

> In D35989#1360151 <https://reviews.llvm.org/D35989#1360151>, @sanjoy wrote:
>
> > In D35989#901470 <https://reviews.llvm.org/D35989#901470>, @mkazantsev wrote:
> >
> > > Re-requesting review per Daniil's comment above. @sanjoy , is this overhead acceptable? I can come up with an alternative maps-based solution if it is not (i.e. only calculate this size for those SCEVs where we need it).
> >
> >
> > Do you have an estimate of what is the max memory consumption when bootstrapping?  I'm pretty sure 0.8M is a very small fraction of the total memory consumption, but would be nice to be sure.
>
>
> We were estimating max amount of SCEVs that exist at the same time during a bootstrap compilation

I meant the maximum amount of memory consumed by clang (in a release without asserts build).

Repository:
  rL LLVM

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

https://reviews.llvm.org/D35989