[llvm] [PDB] Fix missing `consumeError` which raise error with asserts enabled (PR #116480)
Alexandre Ganea via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 18 07:50:36 PST 2024
aganea wrote:
> I can't share the PDB that triggers this code path but the changes are pretty conservative and idiomatic with other `consumeError` used in this source file.
Are you able nevertheless to craft a reduced test that exercices this codepath? There's quite a few examples in https://github.com/llvm/llvm-project/tree/main/llvm/test/DebugInfo/PDB (please see the Inputs folder as well). But I wonder if yaml2pdb would forcibly write the publics stream, if that's the issue.
https://github.com/llvm/llvm-project/pull/116480
More information about the llvm-commits
mailing list