[PATCH] D118597: [DebugInfo][InstrRef][NFC] Don't build mappings of values to locations for un-needed values

[Jeremy Morse via Phabricator via llvm-commits]

jmorse added inline comments.


================
Comment at: llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp:323
       auto ValuesPreferredLoc = ValueToLoc.find(Num);
-      if (ValuesPreferredLoc == ValueToLoc.end()) {
+      if (ValuesPreferredLoc == ValueToLoc.end() ||
+          ValuesPreferredLoc->second.isIllegal()) {
----------------
Orlando wrote:
> Hmm... can `ValuesPreferredLoc == ValueToLoc.end()` ever be true now, with this patch?
Good point, I'll remove that when landing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118597