[PATCH] D74706: [WIP][DebugInfo][LiveDebugValues] Index variable location IDs by machine location

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

I'm glad you've recovered!

I was looking at a benchmark for glyph rendering in WebKit. Unfortunately, the file appears to reference headers from an Apple-internal SDK, and I can't find an analogous benchmark in open source. I've stashed the bitcode for now: happy to benchmark future patches against it if requested.

Re: the early-exits [0, 1] you linked to, I don't understand why these exist. It could be a compile-time hack, but I'm not sure about that, because the loop over OpenLocs remains O(n). Do you expect getting rid of the early exits to result in a large increase in VarLocs? I suppose I'd expect a small/modest increase.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74706