[PATCH] D62196: [LiveDebugValues] Close range for previous variable's location when deducing new variable's location

Nikola Prica via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 23 09:15:17 PDT 2019


NikolaPrica marked an inline comment as done.
NikolaPrica added inline comments.


================
Comment at: lib/CodeGen/LiveDebugValues.cpp:712
   VarLocSet &VLS = OutLocs[CurMBB];
   Changed = VLS |= OpenRanges.getVarLocs();
+  // New OutLocs set may be different due to spill, restore or register
----------------
dstenb wrote:
> Can this be `VLS | OpenRanges.getVarLocs();` now?
Unfortunately that returns SparseBitVector object, we can't deduce from that whether changes were made.


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

https://reviews.llvm.org/D62196





More information about the llvm-commits mailing list