[PATCH] D96033: [clang-repl] Land initial infrastructure for incremental parsing

Mon Aug 16 22:16:31 PDT 2021

v.g.vassilev added subscribers: samitolvanen, pcc.
v.g.vassilev added a comment.

In D96033#2948243 <https://reviews.llvm.org/D96033#2948243>, @leonardchan wrote:

>> @leonardchan, @phosek, I am not aware of such flag. Do you know how much memory does LTO + clang-repl consume and would it make sense to ping some of the LTO folks for their advice?
>
> I played around a bit with adding a flag and it turned out to be not as difficult as I thought it would: D108173 <https://reviews.llvm.org/D108173>. I think making this more of an "opt-in/out" tool like clang-staticanalyzer might be a good idea in general since not all downstream users may want to built all clang tools.

I would rather fix the underlying issue. @samitolvanen, @pcc do you know who can help us with debugging excessive memory use when linking clang-repl using LTO?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96033