[PATCH] D159468: [AsmPrinter][DwarfDebug] Skip vars with fragments in different location kinds

Felipe de Azevedo Piovezan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 03:57:32 PDT 2023


fdeazeve added a comment.

In D159468#4646851 <https://reviews.llvm.org/D159468#4646851>, @dblaikie wrote:

> Probably worth a FIXME and a bug filed about supporting different location kinds? (like, really - we can't describe a variable as being partly in memory, and partly in a register? That doesn't sound right to me... I'd have thought we would've had support for that a while ago, but I don't follow the variable location parts of DWARF in as much detail, so maybe it is)

The whole support for entry-value registers being stored in the MF side-table is new, and this multi-location scenario is something I had not anticipated. We'll get there though! But I agree that a FIXME comment is worth it!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159468



More information about the llvm-commits mailing list