Index: llvm/trunk/lib/Support/YAMLTraits.cpp =================================================================== --- llvm/trunk/lib/Support/YAMLTraits.cpp +++ llvm/trunk/lib/Support/YAMLTraits.cpp @@ -160,7 +160,8 @@ MapHNode *MN = dyn_cast(CurrentNode); if (!MN) { - setError(CurrentNode, "not a mapping"); + if (Required || !isa(CurrentNode)) + setError(CurrentNode, "not a mapping"); return false; } MN->ValidKeys.push_back(Key); Index: llvm/trunk/test/tools/yaml2obj/empty-symbols.yaml =================================================================== --- llvm/trunk/test/tools/yaml2obj/empty-symbols.yaml +++ llvm/trunk/test/tools/yaml2obj/empty-symbols.yaml @@ -0,0 +1,10 @@ +# Ensure yaml2obj doesn't error on empty optional mappings, such as Symbols +# RUN: yaml2obj %s -o %t + +!ELF +FileHeader: + Class: ELFCLASS64 + Data: ELFDATA2LSB + Type: ET_DYN + Machine: EM_X86_64 +Symbols: