[llvm] [YAML] Init local set by a single branche (PR #123137)
Vitaly Buka via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 15 15:35:34 PST 2025
vitalybuka wrote:
> OK, so I looked more. I think only on one branch this is not set, that is line 1601 if the condition below is false.
>
> For 1597 branch we should bail out in the 1613 condition below.
it will not be set if:
1. (TypeStr.starts_with("SHT_") || isInteger(TypeStr)) == false
here we want go to switch and fall through to default.
3. IO.mapRequired("Type", Type); fail parsing
it sets error internally, so probably not important what happen next
https://github.com/llvm/llvm-project/pull/123137
More information about the llvm-commits
mailing list