[DynInst_API:] [dyninst/dyninst] 4689d6: Remove dead code in PowerPC codegen (#2261)

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/thaines/cleanup_ppc_generateBranchViaTrap
  Home:   https://github.com/dyninst/dyninst
  Commit: 4689d61edf8d5c2a35110f4e9c7c68e60bc7e27e
      https://github.com/dyninst/dyninst/commit/4689d61edf8d5c2a35110f4e9c7c68e60bc7e27e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-05-28 (Thu, 28 May 2026)

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

  Log Message:
  -----------
  Remove dead code in PowerPC codegen (#2261)


  Commit: 9c0253ff012ab5da15b43180875c4f1ca78928bd
      https://github.com/dyninst/dyninst/commit/9c0253ff012ab5da15b43180875c4f1ca78928bd
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-05-29 (Fri, 29 May 2026)

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

  Log Message:
  -----------
  [dyninstAPI] Clean up generateBranchViaTrap on PowerPC

There was a lot of discussion in
https://github.com/dyninst/dyninst/issues/175 and
https://github.com/dyninst/dyninst/issues/356 about what the correct
thing to do here is. It looks like neither suggestion was ever
implemented. I'm removing the commented-out code to reduce noise.

The "potential strategy" outline is _NOT_ implemented. I'm leaving that
for possible future usage.


Compare: https://github.com/dyninst/dyninst/compare/4689d61edf8d%5E...9c0253ff012a

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