[PATCH] D107823: [DebugInfo][InstrRef] Honour too-much-debug-info cut-outs in InstrRefBasedLDV

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 16 08:33:38 PDT 2021


jmorse updated this revision to Diff 366644.
jmorse added a comment.

Asan picked up that the early-exit I've added to InstrrefBasedLDV means that the cleanup code doesn't run, and various bits of memory don't get deallocated. It's only a small portion of code between the early-exit block and the cleanup code, so I've just wrapped it in an `else` condition.

Without objection I'll land this again tomorrow.


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

https://reviews.llvm.org/D107823

Files:
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
  llvm/lib/CodeGen/LiveDebugValues/LiveDebugValues.cpp
  llvm/lib/CodeGen/LiveDebugValues/LiveDebugValues.h
  llvm/lib/CodeGen/LiveDebugValues/VarLocBasedImpl.cpp
  llvm/test/DebugInfo/MIR/X86/live-debug-values-cutoffs.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D107823.366644.patch
Type: text/x-patch
Size: 10992 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210816/5b0d6f97/attachment.bin>


More information about the llvm-commits mailing list