[PATCH] D87357: [SelectionDAG][DebugInfo] Use entry-values to recover parameters values

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

I think this looks mostly good.



================
Comment at: llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp:710
 
   if (SD->isInvalidated()) {
+    Register Reg = 0;
----------------
Can you add a comment that explains what's happening here?


================
Comment at: llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp:718
+    if (EntryValue && ValueMap) {
+      auto VMI = ValueMap->find(EntryValue);
+      if (VMI != ValueMap->end()) {
----------------
Would ValueMap.lookup() work here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87357