[PATCH] D116821: [DebugInfo][InstrRef] Move instr-ref controlling flag out of TargetOptions

[Jeremy Morse via Phabricator via llvm-commits]

jmorse added a comment.

In D116821#3279890 <https://reviews.llvm.org/D116821#3279890>, @jmorse wrote:

> An easy way out would be to set a hard limit on the number of spill slots being tracked, on the (to me) fairly reasonable principle of "If your working set is over (say) 1000 Values, you probably autogenerated this code and probably won't be debugging it by hand", much like existing cut-offs in LiveDebugValues. I instinctively don't want to add arbitrary limits though.

(... or maybe for large inputs reduce the (currently highly detailed) quality of information about stack slots, down to the level VarLocBasedLDV works at. You probably don't care about individual subregister-fields within slots if you have so many Values. Quickly hacking shows this reduces memory blow-up to 10%, time to 15%).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116821