[llvm] [DebugInfo] Swap 'Unit' and 'Type' positions in DISubprogram. (PR #96474)
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 2 12:53:02 PDT 2024
dwblaikie wrote:
This change will need test coverage - it might be worth keeping the existing tests around and turning them into code generation tests, to check we can generate DWARF even when we violate these verifier checks?
And we'll need a new test for the intended functionality that motivated you to look into this issue in the first place? (the missing language code/incorrect bounds checking issue? Though I realize it's sort of a non-issue once there's no bounds checking at all anyway... )
https://github.com/llvm/llvm-project/pull/96474
More information about the llvm-commits
mailing list