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

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 08:55:19 PDT 2019


aprantl added inline comments.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfDebug.cpp:1355
 
+    // Check whether location list builder managed to merge all locations to one
+    // that is valid throughout the variable's scope. If so, produce single
----------------
Tiny nit: Can we use the actual class name instead of paraphrasing it as  "location list builder"?


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

https://reviews.llvm.org/D61600





More information about the llvm-commits mailing list