[all-commits] [llvm/llvm-project] 14c2fa: [llvm-pdbutil] Fix register enum field dumping/par...

[Nikita via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 14c2fa27aef3db4fc657241bde18995a111a4858
      https://github.com/llvm/llvm-project/commit/14c2fa27aef3db4fc657241bde18995a111a4858
  Author: Nikita <69168929+nikitalita at users.noreply.github.com>
  Date:   2025-06-02 (Mon, 02 Jun 2025)

  Changed paths:
    A llvm/test/tools/llvm-pdbutil/Inputs/register-records.yaml
    A llvm/test/tools/llvm-pdbutil/register-records.test
    M llvm/tools/llvm-pdbutil/PdbYaml.cpp
    M llvm/tools/llvm-pdbutil/PdbYaml.h
    M llvm/tools/llvm-pdbutil/YAMLOutputStyle.cpp

  Log Message:
  -----------
  [llvm-pdbutil] Fix register enum field dumping/parsing (#82299)

This fixes a bug where parsing PDBs with usages of register enums were
asserting.

The main problem is that printing out the code view register enums are
taken care of here:

https://github.com/nikitalita/llvm-project/blob/e4888a92402f53000a3a5e79d3792c034fc2f343/llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp#L152

Which requires a COFF::header in the IO context for the machine type,
which we didn't have when dumping a pdb or parsing a yaml file. So, we
make a fake one with the machine type.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications