[PATCH] D105029: [Debug-info][InstrRef] Avoid un-necessary ordering in debug value-substitution records

[Jeremy Morse via Phabricator via llvm-commits]

jmorse created this revision.
jmorse added reviewers: djtodoro, Orlando, StephenTozer.
Herald added subscribers: mgrang, hiraditya.
jmorse requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

As suggested by @djtodoro in https://reviews.llvm.org/D88891#2843425 , there's a std::map in MachineFunction that doesn't need to be ordered through most of compilation, only at the end. It can be replaced with a vector that gets ordered when needed, which is what this patch does.

NFCI, and only affects the experimental instruction-referencing work that's currently hidden behind a flag.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D105029

Files:
  llvm/include/llvm/CodeGen/MachineFunction.h
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
  llvm/lib/CodeGen/MIRPrinter.cpp
  llvm/lib/CodeGen/MachineFunction.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105029.354889.patch
Type: text/x-patch
Size: 6034 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210628/78f7a787/attachment.bin>