[llvm] [IPO] Avoid crash when DISubprogram has no type (PR #193420)
Jiang Ning via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 23 02:20:54 PDT 2026
JiangNingHX wrote:
> If we're not aware of any frontend that produces this IR, then I would prefer to reject it in the verifier, so we don't have to deal with it everywhere.
Thanks, I followed up on #186557 and confirmed that the reproducer is crafted/mutated IR, not frontend-emitted IR.
So I agree the current PR is framed too strongly. Given the current LangRef requirement for `DISubprogram::type`, verifier rejection seems like the better direction.
Unless there are objections, I'll stop pursuing this PR in its current form and follow up with a verifier-side fix instead.
https://github.com/llvm/llvm-project/pull/193420
More information about the llvm-commits
mailing list