[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