[PATCH] D77576: [globalisel] Add lost debug locations verifier

[Jessica Paquette via Phabricator via llvm-commits]

paquette added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/Legalizer.cpp:369
+                                      /*MBB=*/&*MF.begin());
+    R << "lost " << std::to_string(LocObserver.getNumLostDebugLocs())
+      << " debug locations during pass";
----------------
Can this use NV?

E.g. something like

```
R << "lost " << NV("NumLostDebugLocs", LocObserver.getNumLostDebugLocs()) << " debug locations during pass";
```

I think that would make optimization records easier to parse, if you choose to save them.

It would also be nice to have a testcase for the remarks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77576