[DynInst_API:] [dyninst/dyninst] f8c658: Update maxInstructionLength, m_size


Date: Thu, 06 Nov 2025 12:56:09 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] f8c658: Update maxInstructionLength, m_size
  Branch: refs/heads/thaines/refactor_Instruction_raw
  Home:   https://github.com/dyninst/dyninst
  Commit: f8c65830bbb65279d33085c756fa60359ac65b10
      https://github.com/dyninst/dyninst/commit/f8c65830bbb65279d33085c756fa60359ac65b10
  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: 981e73b8c5861ea5606eb8ddc5c9524999b0d97b
      https://github.com/dyninst/dyninst/commit/981e73b8c5861ea5606eb8ddc5c9524999b0d97b
  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: 0a4d140dac12b51143cefce50d1568244473ad08
      https://github.com/dyninst/dyninst/commit/0a4d140dac12b51143cefce50d1568244473ad08
  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: 9e1382570cbae3e2f3597ec4b62a6c2b649c3ce5
      https://github.com/dyninst/dyninst/commit/9e1382570cbae3e2f3597ec4b62a6c2b649c3ce5
  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: 4fc9a2cc6ab14faadf1374095c1ccb77a7899a64
      https://github.com/dyninst/dyninst/commit/4fc9a2cc6ab14faadf1374095c1ccb77a7899a64
  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: 2c3f8da06dedd65a909db3b2b229e5d65c30034e
      https://github.com/dyninst/dyninst/commit/2c3f8da06dedd65a909db3b2b229e5d65c30034e
  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: 1d2971d1007ac5547b62a32631f5e3f33ecb7644
      https://github.com/dyninst/dyninst/commit/1d2971d1007ac5547b62a32631f5e3f33ecb7644
  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: 03fa7c2ade77471fb9565ca760bd1ce2b87f82c4
      https://github.com/dyninst/dyninst/commit/03fa7c2ade77471fb9565ca760bd1ce2b87f82c4
  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: e8b66e812c9402ed962a07415d4fce2bd61de610
      https://github.com/dyninst/dyninst/commit/e8b66e812c9402ed962a07415d4fce2bd61de610
  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: 72c6053cb4a2fa34f9c56d7db97d939c07cf40d9
      https://github.com/dyninst/dyninst/commit/72c6053cb4a2fa34f9c56d7db97d939c07cf40d9
  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: ec8696646930c14771c108b463aa8dd944accd97
      https://github.com/dyninst/dyninst/commit/ec8696646930c14771c108b463aa8dd944accd97
  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: 04699056aea1a57fff708201f230d9df511d497e
      https://github.com/dyninst/dyninst/commit/04699056aea1a57fff708201f230d9df511d497e
  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: c8840f30934826db2f400e9689104c657130e776
      https://github.com/dyninst/dyninst/commit/c8840f30934826db2f400e9689104c657130e776
  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: 4d7a73281ac63aa3919b2bc8886956e4996c4c21
      https://github.com/dyninst/dyninst/commit/4d7a73281ac63aa3919b2bc8886956e4996c4c21
  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/24e6862617f9...4d7a73281ac6

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