[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#3254088 <https://reviews.llvm.org/D116821#3254088>, @jmorse wrote:

> ... although when I tested this prior to landing [0] there weren't any double-digit instruction-count regressions, I'll check whether I've missed a patch or something.

^ Ignore that, turns out I can't read, this pretty much matches what I got pre-landing.

tl;dr: this is working as intended -- whether it's a worthwhile trade-off is open to discussion. For testing / profiling examples / opting out, the option is "-mllvm -experimental-debug-variable-locations=false".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116821