[DynInst_API:] [dyninst/dyninst] 5758d3: Clean up dyninstAPI codegen/instrumentation files ...


Date: Tue, 18 Feb 2025 08:05:19 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 5758d3: Clean up dyninstAPI codegen/instrumentation files ...
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 5758d3e5ceb384ab72fd9a2c7492c5b10e236ac4
      https://github.com/dyninst/dyninst/commit/5758d3e5ceb384ab72fd9a2c7492c5b10e236ac4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-18 (Tue, 18 Feb 2025)

  Changed paths:
    M common/CMakeLists.txt
    M common/src/arch-x86.h
    R common/src/arch.h
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/h/BPatch_instruction.h
    M dyninstAPI/h/BPatch_memoryAccess_NP.h
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_binaryEdit.C
    M dyninstAPI/src/BPatch_image.C
    M dyninstAPI/src/BPatch_instruction.C
    M dyninstAPI/src/BPatch_memoryAccess.C
    M dyninstAPI/src/BPatch_memoryAccessAdapter.C
    M dyninstAPI/src/BPatch_parRegion.C
    M dyninstAPI/src/BPatch_point.C
    M dyninstAPI/src/BPatch_process.C
    M dyninstAPI/src/Relocation/Springboard.C
    M dyninstAPI/src/Relocation/Widgets/CFWidget-ppc.C
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/baseTramp.C
    M dyninstAPI/src/baseTramp.h
    M dyninstAPI/src/codegen-aarch64.C
    M dyninstAPI/src/codegen-aarch64.h
    M dyninstAPI/src/codegen-power.C
    M dyninstAPI/src/codegen-power.h
    M dyninstAPI/src/codegen-x86.h
    M dyninstAPI/src/codegen.h
    M dyninstAPI/src/emit-x86.h
    M dyninstAPI/src/image.C
    M dyninstAPI/src/inst-aarch64.C
    M dyninstAPI/src/inst-linux.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-winnt.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/inst.C
    R dyninstAPI/src/instP.h
    M dyninstAPI/src/instPoint.C
    M dyninstAPI/src/instPoint.h
    R dyninstAPI/src/legacy-instruction.h
    M dyninstAPI/src/parRegion.h
    M dyninstAPI/src/parse-aarch64.C
    M dyninstAPI/src/parse-cfg.h
    M dyninstAPI/src/parse-power.C
    M dyninstAPI/src/pdwinnt.C
    M dyninstAPI/src/registerSpace.C
    M parseAPI/src/IA_aarch64.C
    M parseAPI/src/IA_amdgpu.C
    M parseAPI/src/IA_power.C
    M parseAPI/src/InstructionAdapter.C
    M parseAPI/src/Parser-speculative.C
    M proccontrol/src/loadLibrary/codegen-aarch64.C

  Log Message:
  -----------
  Clean up dyninstAPI codegen/instrumentation files (#1891)

* Remove parse_func::isTrueCallInsn

This is only defined for x86 and is never used.

* Remove dyninstAPI/src/instP.h

The functions it declares were either not defined or never used.


* Move x86 codeBuf* typedefs into x86-arch.h

This makes it consistent with the other platforms.

* Remove arch-forward-decl

It's not needed anymore.

* Remove 'arch.h'

It's not needed anymore.

* Remove legacy-instruction.h

As far as I can tell, 'internal_instruction' was never used for
anything dating back to 8da2918a in 2009 during the port to using IAAPI
for parsing.



To unsubscribe from these emails, change your notification settings at https://github.com/dyninst/dyninst/settings/notifications
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 5758d3: Clean up dyninstAPI codegen/instrumentation files ..., Tim Haines <=