[all-commits] [llvm/llvm-project] 449548: [DebugInfo][RemoveDIs] Interpret DPValue objects i...

[Jeremy Morse via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4495485eb6558aac0b0fd81e90b317a8aca30945
      https://github.com/llvm/llvm-project/commit/4495485eb6558aac0b0fd81e90b317a8aca30945
  Author: Jeremy Morse <jeremy.morse at sony.com>
  Date:   2023-11-21 (Tue, 21 Nov 2023)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
    M llvm/test/DebugInfo/X86/arg-dbg-value-list.ll
    M llvm/test/DebugInfo/X86/dbg-empty-metadata-lowering.ll
    M llvm/test/DebugInfo/X86/dbg-val-list-dangling.ll
    M llvm/test/DebugInfo/X86/dbg-val-list-undef.ll
    M llvm/test/DebugInfo/X86/dbg-value-arg-movement.ll
    M llvm/test/DebugInfo/X86/sdag-dbgvalue-phi-use-1.ll
    M llvm/test/DebugInfo/X86/sdag-dbgvalue-phi-use-2.ll
    M llvm/test/DebugInfo/X86/sdag-dbgvalue-phi-use-3.ll
    M llvm/test/DebugInfo/X86/sdag-dbgvalue-phi-use-4.ll
    M llvm/test/DebugInfo/X86/sdag-dbgvalue-ssareg.ll
    M llvm/test/DebugInfo/X86/sdag-ir-salvage.ll
    M llvm/tools/llc/llc.cpp

  Log Message:
  -----------
  [DebugInfo][RemoveDIs] Interpret DPValue objects in SelectionDAG (#72253)

DPValues are the non-intrinsic replacements for dbg.values, and when an
IR function is converted by SelectionDAG we need to convert the variable
location information in the same way. Happily all the information is in
the same format, it's just stored in a slightly different object,
therefore this patch refactors a few things to store the set of
{Variable,Expr,DILocation,Location} instead of just a pointer to a
DbgValueInst.

This also adds a hook in llc that's much like the one I've added to opt
in PR #71937, allowing tests to optionally ask for the use RemoveDIs
mode if support for it is built into the compiler.

I've added that flag to a variety of SelectionDAG debug-info tests to
ensure that we get some coverage on the RemoveDIs / debug-info-iterator
buildbot.