[DynInst_API:] [dyninst/dyninst] 246c10: Update maxInstructionLength, m_size


Date: Thu, 06 Nov 2025 20:10:19 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 246c10: Update maxInstructionLength, m_size
  Branch: refs/heads/thaines/refactor_Instruction_raw
  Home:   https://github.com/dyninst/dyninst
  Commit: 246c103230e3694765b1e9cf1f3eb460ce34f44b
      https://github.com/dyninst/dyninst/commit/246c103230e3694765b1e9cf1f3eb460ce34f44b
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/h/Instruction.h

  Log Message:
  -----------
  Update maxInstructionLength, m_size

15 bytes should be the longest instruction we know of. By reducing it
to 15 and making 'm_size' a 'uint8_t', we can pack the m_RawInsn into
a 'std::array<uint8_t, maxInstructionLength>' and keep the overhead to
just 8 bytes.


  Commit: c5f15d272f547bf7023c0eed2716b5970a2e22f0
      https://github.com/dyninst/dyninst/commit/c5f15d272f547bf7023c0eed2716b5970a2e22f0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/h/Instruction.h

  Log Message:
  -----------
  Replace raw_instruction_t with std::array<uint8_t, maxInstructionLength>


  Commit: 6af6ec9fc5d86f23e6b819b63d34fd71146f9898
      https://github.com/dyninst/dyninst/commit/6af6ec9fc5d86f23e6b819b63d34fd71146f9898
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Update copyRaw


  Commit: 9326b7a21df238669786dc6b3778d283a4551dae
      https://github.com/dyninst/dyninst/commit/9326b7a21df238669786dc6b3778d283a4551dae
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/h/Instruction.h

  Log Message:
  -----------
  Update operator<


  Commit: 7d3f2944a73cd81d4f09a2638ef7d86b4c76ba3c
      https://github.com/dyninst/dyninst/commit/7d3f2944a73cd81d4f09a2638ef7d86b4c76ba3c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/h/Instruction.h

  Log Message:
  -----------
  Update operator==


  Commit: eb9597a2bb7725840665b530facbd1a796d55c37
      https://github.com/dyninst/dyninst/commit/eb9597a2bb7725840665b530facbd1a796d55c37
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

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

  Log Message:
  -----------
  Remove copy assignment


  Commit: bfcb8b3603b7c5bff1ed0ed69340bacee9949a31
      https://github.com/dyninst/dyninst/commit/bfcb8b3603b7c5bff1ed0ed69340bacee9949a31
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

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

  Log Message:
  -----------
  Remove copy ctor


  Commit: 3f6d76adcbb4940ec44c0c84666beda8471d89be
      https://github.com/dyninst/dyninst/commit/3f6d76adcbb4940ec44c0c84666beda8471d89be
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

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

  Log Message:
  -----------
  Use default dtor


  Commit: 07cff0dc703dbd4325f3ad5bab52587825cf57e1
      https://github.com/dyninst/dyninst/commit/07cff0dc703dbd4325f3ad5bab52587825cf57e1
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Update default ctor

This could use default class member initialization, instead. I opted to
do it this way since none of the other members is default-initialized
currently.


  Commit: ac9d97b6dfc333988286ec516bd72b22d491d8a7
      https://github.com/dyninst/dyninst/commit/ac9d97b6dfc333988286ec516bd72b22d491d8a7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Update ctor


  Commit: ea26b0d8a94e32f45b86a159acc76c258390b6b5
      https://github.com/dyninst/dyninst/commit/ea26b0d8a94e32f45b86a159acc76c258390b6b5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Update Instruction::ptr()


  Commit: 7a3d931e0155c9e88ccbeacdea04f5a2064e5e65
      https://github.com/dyninst/dyninst/commit/7a3d931e0155c9e88ccbeacdea04f5a2064e5e65
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M instructionAPI/src/Instruction.C

  Log Message:
  -----------
  Update Instruction::rawByte


  Commit: c673aada8a132583abfd546e6cb8567cba6ad9b7
      https://github.com/dyninst/dyninst/commit/c673aada8a132583abfd546e6cb8567cba6ad9b7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

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

  Log Message:
  -----------
  Update aarch64 decoder


  Commit: ee0173ade425096f7f82ee1982213f0fb2b7b2a5
      https://github.com/dyninst/dyninst/commit/ee0173ade425096f7f82ee1982213f0fb2b7b2a5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

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

  Log Message:
  -----------
  Update ppc decoder


Compare: https://github.com/dyninst/dyninst/compare/4d7a73281ac6...ee0173ade425

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] 246c10: Update maxInstructionLength, m_size, Tim Haines <=