[all-commits] [llvm/llvm-project] 6144fc: [NewPM] Print pre-transformation IR name in --prin...

[aeubanks via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6144fc2da1b87dc64ff887d73b60f7708f5cb0a4
      https://github.com/llvm/llvm-project/commit/6144fc2da1b87dc64ff887d73b60f7708f5cb0a4
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2021-07-20 (Tue, 20 Jul 2021)

  Changed paths:
    M llvm/include/llvm/Passes/StandardInstrumentations.h
    M llvm/lib/Passes/StandardInstrumentations.cpp
    M llvm/test/Other/loop-deletion-printer.ll
    M llvm/test/Other/scc-deleted-printer.ll

  Log Message:
  -----------
  [NewPM] Print pre-transformation IR name in --print-after-all

Sometimes a transformation can change the name of some IR (e.g. an SCC
with functions added/removed). This can be confusing when debug logging
doesn't match the post-transformation name. The specific example I came
across was that --print-after-all said the inliner was working on an SCC
that only contained one function, but calls in multiple functions were
getting inlined. After all inlining, the current SCC only contained one
function.

Piggyback off of the existing logic to handle invalidated IR +
--print-module-scope. Simply always store the IR description and use
that.

Reviewed By: jamieschmeiser

Differential Revision: https://reviews.llvm.org/D106290