[DynInst_API:] [dyninst/dyninst] 29ea6f: Simplify invalid Operation construction


Date: Wed, 18 Mar 2026 12:29:22 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 29ea6f: Simplify invalid Operation construction
  Branch: refs/heads/thaines/extract_x86_Operation_ctor
  Home:   https://github.com/dyninst/dyninst
  Commit: 29ea6f6777affaf3e47dde6baff9f0d1cc5a81f6
      https://github.com/dyninst/dyninst/commit/29ea6f6777affaf3e47dde6baff9f0d1cc5a81f6
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-18 (Wed, 18 Mar 2026)

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

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


  Commit: 291aace21ff036776c43dd042a1acfaa5f309770
      https://github.com/dyninst/dyninst/commit/291aace21ff036776c43dd042a1acfaa5f309770
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-18 (Wed, 18 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: 65fb33160f08b4742ea3c31111ebbec9671b904a
      https://github.com/dyninst/dyninst/commit/65fb33160f08b4742ea3c31111ebbec9671b904a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-18 (Wed, 18 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: f7566eac54ddba254264af6e19fd7a7b11fbec0e
      https://github.com/dyninst/dyninst/commit/f7566eac54ddba254264af6e19fd7a7b11fbec0e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-18 (Wed, 18 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/7725085b64a3...f7566eac54dd

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