[PATCH] D110332: [DebugInfo] Limit the size of DIExpressions that we will salvage up to

[Sanjay Patel via Phabricator via llvm-commits]

spatel edited reviewers, added: nikic; removed: spatel.
spatel added a comment.

Thanks for working on this! I don't know anything about debug info/salvaging implementation, so I can't review the patch, but I hope someone else will have a look.

Could this reduce compile time in this config:
https://llvm-compile-time-tracker.com/?config=NewPM-ReleaseLTO-g&stat=instructions ?

If so, it would be good to know that number before committing - maybe tune the MaxExpressionSize setting for most benefit based on those benchmarks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110332