[PATCH] D118601: [DebugInfo][InstrRef] Add a max-stack-slots-to-track limitation / cut-out

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 31 07:49:20 PST 2022


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

Fix an off-by-one (The limit should be hit when the number of seen locations is >= the limit, otherwise setting a limit of 0 will never work). Add the inverse test to the MIR file added, to check that stack locations would actually be found if the limit wasn't on.


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

https://reviews.llvm.org/D118601

Files:
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.h
  llvm/test/DebugInfo/MIR/InstrRef/spill-slot-limits.mir
  llvm/unittests/CodeGen/InstrRefLDVTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118601.404532.patch
Type: text/x-patch
Size: 18994 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220131/e6091376/attachment.bin>


More information about the llvm-commits mailing list