[DynInst_API:] [dyninst/dyninst] 59e1a9: Extract instMapping


Date: Wed, 22 Apr 2026 23:40:26 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 59e1a9: Extract instMapping
  Branch: refs/heads/thaines/cleanup_instMapping
  Home:   https://github.com/dyninst/dyninst
  Commit: 59e1a9fabc450f988ae4efdcffb599318e6cd8e7
      https://github.com/dyninst/dyninst/commit/59e1a9fabc450f988ae4efdcffb599318e6cd8e7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_point.C
    M dyninstAPI/src/BPatch_private.h
    M dyninstAPI/src/Relocation/DynInstrumenter.C
    M dyninstAPI/src/Relocation/DynInstrumenter.h
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/dynProcess.h
    M dyninstAPI/src/inst.C
    M dyninstAPI/src/inst.h
    A dyninstAPI/src/instMapping.C
    A dyninstAPI/src/instMapping.h
    M dyninstAPI/src/syscallNotification.C
    M dyninstAPI/src/syscallNotification.h

  Log Message:
  -----------
  Extract instMapping


  Commit: 4e29889581f45c093bba1ce072030bae42db4c48
      https://github.com/dyninst/dyninst/commit/4e29889581f45c093bba1ce072030bae42db4c48
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.C
    M dyninstAPI/src/instMapping.h

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


  Commit: 2848a9f0dc75530d3ab76ec2420a0a06debe6fb1
      https://github.com/dyninst/dyninst/commit/2848a9f0dc75530d3ab76ec2420a0a06debe6fb1
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Don't typedef enums


  Commit: 8ff2eb211b87fd6d4209b05f78b50a3ce1926ee7
      https://github.com/dyninst/dyninst/commit/8ff2eb211b87fd6d4209b05f78b50a3ce1926ee7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Don't use explicit pointer checks


  Commit: e327d0903fd89f8ba2c4503113fdb7fd5bd1c872
      https://github.com/dyninst/dyninst/commit/e327d0903fd89f8ba2c4503113fdb7fd5bd1c872
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove dtor

The compiler-provided one is sufficient and now allows compiler to
generate move and copy operations.


  Commit: 60d988aef54d56a84db76b3b57768dff3c670ae6
      https://github.com/dyninst/dyninst/commit/60d988aef54d56a84db76b3b57768dff3c670ae6
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove extraneous access specifier


  Commit: 832ecafdc21312bd905eeb289c2636aa2c56507c
      https://github.com/dyninst/dyninst/commit/832ecafdc21312bd905eeb289c2636aa2c56507c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Reduce string copies


  Commit: a0d76f5656f5454c3a04c178dcc4a1cf30e44bef
      https://github.com/dyninst/dyninst/commit/a0d76f5656f5454c3a04c178dcc4a1cf30e44bef
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove explicit namespace prefix


  Commit: d649ac25f79c5e3227ad4a9c5323ff1792df9556
      https://github.com/dyninst/dyninst/commit/d649ac25f79c5e3227ad4a9c5323ff1792df9556
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Make is_MTonly const


  Commit: eabc1b4c41933efbc346222f49c3a8fd36cae61c
      https://github.com/dyninst/dyninst/commit/eabc1b4c41933efbc346222f49c3a8fd36cae61c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove unused ctors


  Commit: 28328961c23d00f28ce83a6c006223a14c239fd9
      https://github.com/dyninst/dyninst/commit/28328961c23d00f28ce83a6c006223a14c239fd9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/inst.h
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Extract instrumentation location macros


  Commit: fdf914be21dc48508d765d363e386e3533a160c1
      https://github.com/dyninst/dyninst/commit/fdf914be21dc48508d765d363e386e3533a160c1
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove unused markAs_MTonly


  Commit: c4559e32720401be9d6694051bf857f5f265995e
      https://github.com/dyninst/dyninst/commit/c4559e32720401be9d6694051bf857f5f265995e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Use in-class initializers for members not set in ctor


  Commit: 14a90f9c35fd3c0f05034d56390d139aad10a529
      https://github.com/dyninst/dyninst/commit/14a90f9c35fd3c0f05034d56390d139aad10a529
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/instMapping.C
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove mt_only

It's always 'false', so just remove the check.


  Commit: 54d28805f3b2d04b6ae44292dbcd26a83307c356
      https://github.com/dyninst/dyninst/commit/54d28805f3b2d04b6ae44292dbcd26a83307c356
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove library name

It's always empty.


  Commit: d3593e420ece1fe599aa3f169fb6f6ab2a9d5ae0
      https://github.com/dyninst/dyninst/commit/d3593e420ece1fe599aa3f169fb6f6ab2a9d5ae0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.C
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove 'when'

It's never used.


  Commit: dc3bb02761bb70529b356d684e8ee9673c3c4268
      https://github.com/dyninst/dyninst/commit/dc3bb02761bb70529b356d684e8ee9673c3c4268
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/instMapping.C
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Remove 'order'

It's always set to orderLastAtPoint, so the check in PCProcess::installInstrRequests
is always false.


  Commit: 1746c61e6336179b9a0a3e37414892ddfaea7ec4
      https://github.com/dyninst/dyninst/commit/1746c61e6336179b9a0a3e37414892ddfaea7ec4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    R dyninstAPI/src/instMapping.C
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Inline "fork" ctor


  Commit: 11d1b8eaff54910a7002e6ccaf693f4c0283794f
      https://github.com/dyninst/dyninst/commit/11d1b8eaff54910a7002e6ccaf693f4c0283794f
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    R dyninstAPI/src/inst.C

  Log Message:
  -----------
  Remove inst.C

It's no longer needed.


  Commit: 24aeded3ecc5c9529424fed0e8fff1e3e770564f
      https://github.com/dyninst/dyninst/commit/24aeded3ecc5c9529424fed0e8fff1e3e770564f
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-23 (Thu, 23 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

  Log Message:
  -----------
  Move 'args' to init list for "fork" ctor


Compare: https://github.com/dyninst/dyninst/compare/cdcbbfc238cd...24aeded3ecc5

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] 59e1a9: Extract instMapping, Tim Haines <=