[DynInst_API:] [dyninst/dyninst] 83948d: Simplify invalid Operation construction


Date: Mon, 16 Mar 2026 13:34:29 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 83948d: Simplify invalid Operation construction
  Branch: refs/heads/thaines/extract_x86_Operation_ctor
  Home:   https://github.com/dyninst/dyninst
  Commit: 83948db7198da614db7d191e7d6cb2b9124f1449
      https://github.com/dyninst/dyninst/commit/83948db7198da614db7d191e7d6cb2b9124f1449
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-16 (Mon, 16 Mar 2026)

  Changed paths:
    M instructionAPI/src/InstructionDecoder-x86.C

  Log Message:
  -----------
  Simplify invalid Operation construction


  Commit: 9b717c4cff39405ccb19feef46e146f088643dd8
      https://github.com/dyninst/dyninst/commit/9b717c4cff39405ccb19feef46e146f088643dd8
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-16 (Mon, 16 Mar 2026)

  Changed paths:
    M instructionAPI/h/Operation_impl.h
    M instructionAPI/src/InstructionDecoder-x86.C
    M instructionAPI/src/Operation.C

  Log Message:
  -----------
  Extract ia32-specific construction into its decoder


  Commit: 3580413f472d35a0f04cb82440cf5fa29c327f58
      https://github.com/dyninst/dyninst/commit/3580413f472d35a0f04cb82440cf5fa29c327f58
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-16 (Mon, 16 Mar 2026)

  Changed paths:
    M instructionAPI/src/Operation.C

  Log Message:
  -----------
  Move addrWidth calculation out of ctor

The other place it's calculated is in the x86 decoder.


  Commit: 7725085b64a3a45f569812ce2326301c263d2e4c
      https://github.com/dyninst/dyninst/commit/7725085b64a3a45f569812ce2326301c263d2e4c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-16 (Mon, 16 Mar 2026)

  Changed paths:
    M instructionAPI/h/Operation_impl.h
    M instructionAPI/src/Operation.C

  Log Message:
  -----------
  Inline ctor


Compare: https://github.com/dyninst/dyninst/compare/83948db7198d%5E...7725085b64a3

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] 83948d: Simplify invalid Operation construction, Tim Haines <=