[DynInst_API:] [dyninst/dyninst] d4e535: [dyninstAPI] Clean up emitLoadPreviousStackFrameRe...


Date: Mon, 22 Jun 2026 21:11:42 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] d4e535: [dyninstAPI] Clean up emitLoadPreviousStackFrameRe...
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: d4e535ebda44e239d2e15f72e28fc7cd846baf9f
      https://github.com/dyninst/dyninst/commit/d4e535ebda44e239d2e15f72e28fc7cd846baf9f
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-23 (Tue, 23 Jun 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/ASTs/operandAST.C
    R dyninstAPI/src/inst-aarch64.C
    M dyninstAPI/src/inst-amdgpu.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/registerSpace/registerSpace.C
    M dyninstAPI/src/registerSpace/registerSpace.h

  Log Message:
  -----------
  [dyninstAPI] Clean up emitLoadPreviousStackFrameRegister (#2297)

* Remove commented-out call in operandAST::generateCode_phase2

It was commented out by f05f5b5290c622 in 2008.

* Remove dead preprocessor branch in registerSpace::readProgramRegister

* Convert to runtime selection

* 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.

* Merge implementations

* Remove inst-aarch64.C

It's no longer needed.

* Remove unused size param from readProgramRegister



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] d4e535: [dyninstAPI] Clean up emitLoadPreviousStackFrameRe..., Tim Haines <=