[DynInst_API:] [dyninst/dyninst] 43d1e5: Simplify use of DYNINST_EXPORT


Date: Sun, 15 Mar 2026 17:35:35 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 43d1e5: Simplify use of DYNINST_EXPORT
  Branch: refs/heads/thaines/instruction_containers
  Home:   https://github.com/dyninst/dyninst
  Commit: 43d1e5e830eccbdd7bcf907f7f67a4565d763c24
      https://github.com/dyninst/dyninst/commit/43d1e5e830eccbdd7bcf907f7f67a4565d763c24
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Simplify use of DYNINST_EXPORT


  Commit: e559dc5075babe416ed39b3ec3c1d695bb4f00c4
      https://github.com/dyninst/dyninst/commit/e559dc5075babe416ed39b3ec3c1d695bb4f00c4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Don't store the ArchSpecificFormatter.


  Commit: 7d7229f5bf62654277bbbcdf469a7e7fe2016d33
      https://github.com/dyninst/dyninst/commit/7d7229f5bf62654277bbbcdf469a7e7fe2016d33
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Clean up includes in Instruction.C


  Commit: d4ffc9bc46890b88d13fc212de189b70972c4851
      https://github.com/dyninst/dyninst/commit/d4ffc9bc46890b88d13fc212de189b70972c4851
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Remove namespace directive in Instruction.C


  Commit: 7361680254d88c4bcd54cd7d63d9210523e17138
      https://github.com/dyninst/dyninst/commit/7361680254d88c4bcd54cd7d63d9210523e17138
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Define isLegalInsn() in terms of isValid()


  Commit: 34ebe83dd05633d1b55d1686b755a27f3bfc084a
      https://github.com/dyninst/dyninst/commit/34ebe83dd05633d1b55d1686b755a27f3bfc084a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Don't store m_Valid

It's cheap to compute.


  Commit: fe333921211042b7153a6b5f17104e8e8eb547bb
      https://github.com/dyninst/dyninst/commit/fe333921211042b7153a6b5f17104e8e8eb547bb
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Fix whitespace for ctors


  Commit: b511faabaeaa64e6ac05e659bdef6c7a6bf2d483
      https://github.com/dyninst/dyninst/commit/b511faabaeaa64e6ac05e659bdef6c7a6bf2d483
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Use in-class initializers


  Commit: 2b664dbcf08047951fc4c4ff98550dfca091fa05
      https://github.com/dyninst/dyninst/commit/2b664dbcf08047951fc4c4ff98550dfca091fa05
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Modernize loops


  Commit: 1f70f35df9cd76fe6a3382932501f4df0ae047f4
      https://github.com/dyninst/dyninst/commit/1f70f35df9cd76fe6a3382932501f4df0ae047f4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Use explicitly-defaulted default ctor

Members now use in-class initializers.


  Commit: b4a6c0b134fdb8762dfbade892223f3b2173ccc5
      https://github.com/dyninst/dyninst/commit/b4a6c0b134fdb8762dfbade892223f3b2173ccc5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Remove non-const getEncodedOperation()

The only reason the non-const getOperation() exists is because
Operation::isRead isn't const. That will get fixed when x86 migrates to
Capstone because SetUpNonOperandData() will go away.


  Commit: 265eef1c3474fcd1b3d764b25b944219f67df6a3
      https://github.com/dyninst/dyninst/commit/265eef1c3474fcd1b3d764b25b944219f67df6a3
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-15 (Sun, 15 Mar 2026)

  Changed paths:
    M instructionAPI/doc/API/Instruction.tex

  Log Message:
  -----------
  Update docs for Instruction::get(Read,Write)Set


Compare: https://github.com/dyninst/dyninst/compare/684f1b20fceb...265eef1c3474

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] 43d1e5: Simplify use of DYNINST_EXPORT, Tim Haines <=