[all-commits] [llvm/llvm-project] a45311: [DebugInfo][RemoveDIs] Extend intrinsic-conversion...

[Jeremy Morse via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a4531108da358500939af95b53794591432aaf74
      https://github.com/llvm/llvm-project/commit/a4531108da358500939af95b53794591432aaf74
  Author: Jeremy Morse <jeremy.morse at sony.com>
  Date:   2024-02-06 (Tue, 06 Feb 2024)

  Changed paths:
    M llvm/lib/Transforms/Utils/Debugify.cpp
    M llvm/test/Transforms/Util/Debugify/loc-only-original-mode.ll

  Log Message:
  -----------
  [DebugInfo][RemoveDIs] Extend intrinsic-conversion in debugify (#80861)

A while back the entry/exit points of debugify were instrumented with
conversion functions to/from non-intrinsic-form debug-info. This is the
path of least resistance to incrementally converting parts of LLVM to
use the new format. However, it turns out that debugify registers
callbacks with the pass manager and can be fed non-intrinsic form
debug-info. Thus: this patch wraps each of the four major debugify
functions with the convertion utilities, and extends test coverage to a
test that exposes this problem.

(An alternative would be to put this code in the callback lambdas, but
then it would be fighting pass manager abstractions of what type the IR
has).

Handily debugify has been designed to record the /meaning/ of debug-info
rather than take pointers to intrinsics and the like, so the storage
mechanism for debug-info is transparent to it!