[DynInst_API:] [dyninst/dyninst] c068b8: Move getArchRegFromAbstractReg into dwarfFrameParser

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/thaines/MachRegister_getArchReg
  Home:   https://github.com/dyninst/dyninst
  Commit: c068b81811e0f87b1e15cbb1b04588c536dd5eea
      https://github.com/dyninst/dyninst/commit/c068b81811e0f87b1e15cbb1b04588c536dd5eea
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-12 (Wed, 12 Feb 2025)

  Changed paths:
    M common/h/registers/MachRegister.h
    M common/src/registers/MachRegister.C
    M dwarf/src/dwarfFrameParser.C

  Log Message:
  -----------
  Move getArchRegFromAbstractReg into dwarfFrameParser

This is the only place it's used. This change also

1) changes the name
2) does the architecture check at runtime instead of compile time
3) doesn't crash for non-aarch64 platforms


  Commit: a109e7e8667aa9958e239e07c954366ea3411162
      https://github.com/dyninst/dyninst/commit/a109e7e8667aa9958e239e07c954366ea3411162
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-12 (Wed, 12 Feb 2025)

  Changed paths:
    M common/h/registers/MachRegister.h
    M common/src/registers/MachRegister.C
    M proccontrol/src/arm_process.C

  Log Message:
  -----------
  Move getArchReg

It's only ever used in arm_process.C.


Compare: https://github.com/dyninst/dyninst/compare/c068b81811e0%5E...a109e7e8667a

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