[PATCH] D70350: [DWARF] Allow cross-CU references of subprogram definitions
Vedant Kumar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 7 14:06:30 PST 2020
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
More information about the llvm-commits
mailing list