[PATCH] D61600: [DebugInfo] More precise variable range for stack locations

[David Stenberg via Phabricator via llvm-commits]

dstenb added a comment.

Thanks a lot! This "location list -> single location description" detection seems more robust, and should help us moving towards supporting rewrites of single-entry location lists containing fragments also.

Nit: Perhaps we should move the Value class out of DebugLocEntry now that we use it for DbgVariable also? However, I don't have any good ideas on in where to place it (e.g. which header file). I think we can land this without doing such a refactoring though.



================
Comment at: lib/CodeGen/AsmPrinter/DwarfDebug.h:114
 /// location changes over time use \a DebugLocListIndex, while those with a
-/// single instruction use \a MInsn and (optionally) a single entry of \a Expr.
+/// single instruction use \a ValueLoc and (optionally) a single entry of \a Expr.
 ///
----------------
Nit: Perhaps change //single instruction// -> //single location//?


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

https://reviews.llvm.org/D61600