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


Date: Fri, 07 Nov 2025 09:20:32 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] ad1c82: Update maxInstructionLength, m_size
  Branch: refs/heads/thaines/refactor_Instruction_raw
  Home:   https://github.com/dyninst/dyninst
  Commit: ad1c821548acffd47fc851e26d97ab64088cd086
      https://github.com/dyninst/dyninst/commit/ad1c821548acffd47fc851e26d97ab64088cd086
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-07 (Fri, 07 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: fc06ebf42db073dde1faafcbf48162c852c6e1b6
      https://github.com/dyninst/dyninst/commit/fc06ebf42db073dde1faafcbf48162c852c6e1b6
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-07 (Fri, 07 Nov 2025)

  Changed paths:
    M instructionAPI/h/Instruction.h

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


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

  Changed paths:
    M instructionAPI/src/Instruction.C

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


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

  Changed paths:
    M instructionAPI/h/Instruction.h

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


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

  Changed paths:
    M instructionAPI/h/Instruction.h

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


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

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

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


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

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

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


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

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

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


  Commit: 9d50ebcbd7278647fe4ee72265e5606d2b488c6a
      https://github.com/dyninst/dyninst/commit/9d50ebcbd7278647fe4ee72265e5606d2b488c6a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-07 (Fri, 07 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: 2b7f2f0da931c2eec7e93149b171b141c5c455b3
      https://github.com/dyninst/dyninst/commit/2b7f2f0da931c2eec7e93149b171b141c5c455b3
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-11-07 (Fri, 07 Nov 2025)

  Changed paths:
    M instructionAPI/src/Instruction.C

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


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

  Changed paths:
    M instructionAPI/src/Instruction.C

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


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

  Changed paths:
    M instructionAPI/src/Instruction.C

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


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

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

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


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

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

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


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

  Changed paths:
    M instructionAPI/h/Instruction.h

  Log Message:
  -----------
  Fix operator==


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

  Changed paths:
    M instructionAPI/h/Instruction.h

  Log Message:
  -----------
  Fix operator<


Compare: https://github.com/dyninst/dyninst/compare/b4dea1e7f10a...b365ed4c8069

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