[DynInst_API:] [dyninst/dyninst] 5f9430: Remove commented-out call in operandAST::generateC...


Date: Sat, 20 Jun 2026 22:04:25 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 5f9430: Remove commented-out call in operandAST::generateC...
  Branch: refs/heads/thaines/cleanup_emitLoadPreviousStackFrameRegister
  Home:   https://github.com/dyninst/dyninst
  Commit: 5f94307a838890b466c8f4e687b0f2d1d021c6b0
      https://github.com/dyninst/dyninst/commit/5f94307a838890b466c8f4e687b0f2d1d021c6b0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-20 (Sat, 20 Jun 2026)

  Changed paths:
    M dyninstAPI/src/ASTs/operandAST.C

  Log Message:
  -----------
  Remove commented-out call in operandAST::generateCode_phase2

It was commented out by f05f5b5290c622 in 2008.


  Commit: 1ce3b9c9d27df8f88eab5683fda9effd299ade85
      https://github.com/dyninst/dyninst/commit/1ce3b9c9d27df8f88eab5683fda9effd299ade85
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-20 (Sat, 20 Jun 2026)

  Changed paths:
    M dyninstAPI/src/registerSpace/registerSpace.C

  Log Message:
  -----------
  Remove dead preprocessor branch in registerSpace::readProgramRegister


  Commit: 0699a8b43bb58c8f5b5376818f7e4e9b207dec4a
      https://github.com/dyninst/dyninst/commit/0699a8b43bb58c8f5b5376818f7e4e9b207dec4a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-20 (Sat, 20 Jun 2026)

  Changed paths:
    M dyninstAPI/src/registerSpace/registerSpace.C

  Log Message:
  -----------
  Convert to runtime selection


  Commit: d173dd82b3718b130b4a04b8a8222784c26d4eff
      https://github.com/dyninst/dyninst/commit/d173dd82b3718b130b4a04b8a8222784c26d4eff
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-20 (Sat, 20 Jun 2026)

  Changed paths:
    M dyninstAPI/src/inst-power.C

  Log Message:
  -----------
  Remove ppc implementation

The compile-time selection in registerSpace::readProgramRegister was added
by 66682a53ca6d42 in 2008 without any description as to why this
implementation needed to diverge from the one in readProgramRegister.


  Commit: ec7395a67f88f7f4606c35f41d109a9dca65c0bd
      https://github.com/dyninst/dyninst/commit/ec7395a67f88f7f4606c35f41d109a9dca65c0bd
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-20 (Sat, 20 Jun 2026)

  Changed paths:
    M dyninstAPI/src/inst-aarch64.C
    M dyninstAPI/src/inst-amdgpu.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/registerSpace/registerSpace.C
    M dyninstAPI/src/registerSpace/registerSpace.h

  Log Message:
  -----------
  Merge implementations


  Commit: 98844c0ec3e0815c093e8d616fb9fa32318e9032
      https://github.com/dyninst/dyninst/commit/98844c0ec3e0815c093e8d616fb9fa32318e9032
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-20 (Sat, 20 Jun 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    R dyninstAPI/src/inst-aarch64.C

  Log Message:
  -----------
  Remove inst-aarch64.C

It's no longer needed.


  Commit: f8b46589278bcbafb2fc5e48fa04df7c5d04dea2
      https://github.com/dyninst/dyninst/commit/f8b46589278bcbafb2fc5e48fa04df7c5d04dea2
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-21 (Sun, 21 Jun 2026)

  Changed paths:
    M dyninstAPI/src/ASTs/operandAST.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/registerSpace/registerSpace.C
    M dyninstAPI/src/registerSpace/registerSpace.h

  Log Message:
  -----------
  Remove unused size param from readProgramRegister


Compare: https://github.com/dyninst/dyninst/compare/921eac73f20e...f8b46589278b

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] 5f9430: Remove commented-out call in operandAST::generateC..., Tim Haines <=