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

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 21 11:25:09 PDT 2019


aprantl added inline comments.


================
Comment at: lib/CodeGen/LiveDebugValues.cpp:436
+
+    // Close previous variable's location range.
+    DebugVariable V(DMI->getDebugVariable(),
----------------
 previous variable's -> this variable's previous

Does this do the right thing for two fragments of the same variable where one of them has a longer live range than the other? Or is this a non-issue here?


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

https://reviews.llvm.org/D62196





More information about the llvm-commits mailing list