[llvm] [LiveDebugVariables] Add basic verification (PR #68703)
Jeremy Morse via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 10 16:35:34 PDT 2023
jmorse wrote:
In principle this LGTM, although it's been a long time since I've looked at LiveDebugVariables. As far as I recall, we use `setUnchecked` to do manual maintenance of ranges but they should all coalesce in the end. And it doesn't make sense for there to be a zero-length range of variable location.
This does involve iterating over every range of every variable in the function, which might have compile-time performance implications. Is it worth putting this behind an EXPENSIVE_CHECKS ifdef? (I haven't followed the SlotIndex related problems you've mentioned, I don't know whether it needs running on every compilation)
https://github.com/llvm/llvm-project/pull/68703
More information about the llvm-commits
mailing list