[PATCH] D147506: [DebugLine] save one debug line entry for empty prologue

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 10:55:57 PDT 2023


dblaikie added a comment.

> I think for the motivated case on AIX, we need this to be in the higher level, i.e. in the place where we generate the .loc directives. AIX for now uses non integrated-assembler mode by default, so fixing this inside integrated assembler will not make our issue gone.

In that case, maybe in depth would be to fix the AIX assembler too, if it's producing invalid output for the platform?  But, yeah, if that's out of scope and/or to fix this as  robustly as possible, would be good to get this fixed in LLVM at the assembly emission (somewhere generic for all MCStreamer (ie: above MCAsmStreamer/MCObjectStreamer, but below any user of MCStreamer - so presumably that puts the fix somewhere /in/ MCStreamer, generally speaking (maybe some side utilities, etc, I haven't looked at the API in detail))


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147506



More information about the llvm-commits mailing list