[PATCH] D70350: [DWARF] Allow cross-CU references of subprogram definitions

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

Hey @modocache, thanks for the report and stack trace. It looks like clang crashes because it found a DISubprogram without an associated DIE. I'm not sure how this could happen: the function `constructCallSiteEntryDIEs` should ensure that the associated DIE exists. Are you aware of any other internal debug info-related changes which might break this assumption?

FWIW, after adding an assert that guards against this to `constructCallSiteEntryDIEs`, I could build all of LNT using the ReleaseLTO-g cmake cache without issue. I'll kick off LTO builds of our internal projects to exercise this code path more. Meanwhile, is there any chance you could verify that the existing assert at DwarfCompileUnit.cpp:972 fires in your LTO build (https://github.com/llvm/llvm-project/blob/master/llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp#L972)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70350