[DynInst_API:] [dyninst/dyninst] 1ed8fc: Move MachRegister -> ROSE register conversion into...

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 1ed8fcaea4037693aa8a8b89221d0fa93353a2c9
      https://github.com/dyninst/dyninst/commit/1ed8fcaea4037693aa8a8b89221d0fa93353a2c9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2024-12-18 (Wed, 18 Dec 2024)

  Changed paths:
    M common/h/registers/MachRegister.h
    M common/src/registers/MachRegister.C
    M dataflowAPI/CMakeLists.txt
    M dataflowAPI/rose/RegisterDescriptor.h
    A dataflowAPI/rose/registers/aarch64.h
    A dataflowAPI/rose/registers/amdgpu.h
    A dataflowAPI/rose/registers/convert.C
    A dataflowAPI/rose/registers/convert.h
    A dataflowAPI/rose/registers/ppc32.h
    A dataflowAPI/rose/registers/ppc64.h
    A dataflowAPI/rose/registers/x86.h
    A dataflowAPI/rose/registers/x86_64.h
    M dataflowAPI/src/ExpressionConversionVisitor.C
    M external/rose/amdgpuInstructionEnum.h
    M external/rose/armv8InstructionEnum.h

  Log Message:
  -----------
  Move MachRegister -> ROSE register conversion into dataflowAPI (#1842)

The conversion is only ever used in dataflowAPI, so that's where it
belongs. It also simplifies the conversion by directly creating a
RegisterDescriptor which is the register vocabulary for interacting
with ROSE.

* Add logging for unimplemented architectures

There is no `default` case in the switch, so the compiler will warn
about missing enumerators in which case this code should never be
encountered.



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