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


Date: Wed, 17 Jun 2026 20:34:09 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] e37d92: Remove commented-out call in operandAST::generateC...
  Branch: refs/heads/thaines/cleanup_emitLoadPreviousStackFrameRegister
  Home:   https://github.com/dyninst/dyninst
  Commit: e37d92e8c0600c02c34560ab5253a33f3591f152
      https://github.com/dyninst/dyninst/commit/e37d92e8c0600c02c34560ab5253a33f3591f152
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-17 (Wed, 17 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: bb559954a1deedc6b8ae87a7ab6b8729738cfc58
      https://github.com/dyninst/dyninst/commit/bb559954a1deedc6b8ae87a7ab6b8729738cfc58
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-17 (Wed, 17 Jun 2026)

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

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


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

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

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


  Commit: b0357bd1942c791fa543f991bfe4898951cacde2
      https://github.com/dyninst/dyninst/commit/b0357bd1942c791fa543f991bfe4898951cacde2
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-17 (Wed, 17 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: 01abdc40458dbd8dd79caeab4796c3f52a47b620
      https://github.com/dyninst/dyninst/commit/01abdc40458dbd8dd79caeab4796c3f52a47b620
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-17 (Wed, 17 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: c040ba9b22245c9585c1ff7e573cd2692f3e34b0
      https://github.com/dyninst/dyninst/commit/c040ba9b22245c9585c1ff7e573cd2692f3e34b0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-17 (Wed, 17 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: 921eac73f20e59a8e37507ea3c57bdac2533f0c9
      https://github.com/dyninst/dyninst/commit/921eac73f20e59a8e37507ea3c57bdac2533f0c9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-17 (Wed, 17 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/e37d92e8c060%5E...921eac73f20e

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