[DynInst_API:] [dyninst/dyninst] 6cca57: Extract instMapping

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/thaines/cleanup_instMapping
  Home:   https://github.com/dyninst/dyninst
  Commit: 6cca570d59a6636eec490f3f9628ac7b88f3ee9e
      https://github.com/dyninst/dyninst/commit/6cca570d59a6636eec490f3f9628ac7b88f3ee9e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 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: 67ce881253572fce2566dc3c8ca153b22f516fd8
      https://github.com/dyninst/dyninst/commit/67ce881253572fce2566dc3c8ca153b22f516fd8
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

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

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


  Commit: 5f1649e52f42eb4c3b5109dec592b06b20652b87
      https://github.com/dyninst/dyninst/commit/5f1649e52f42eb4c3b5109dec592b06b20652b87
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


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

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


  Commit: 1f9162eb9d1565ee37b70b5c2bfd79035f501607
      https://github.com/dyninst/dyninst/commit/1f9162eb9d1565ee37b70b5c2bfd79035f501607
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 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: d38f73630bc6aad31ca809c796e997f919b99d1e
      https://github.com/dyninst/dyninst/commit/d38f73630bc6aad31ca809c796e997f919b99d1e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


  Commit: 6a467a95776a3240818a5a0e40cc2c74a739c7a8
      https://github.com/dyninst/dyninst/commit/6a467a95776a3240818a5a0e40cc2c74a739c7a8
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


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

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


  Commit: 5f33930c91791dfb0d13a606ea643f305de7519e
      https://github.com/dyninst/dyninst/commit/5f33930c91791dfb0d13a606ea643f305de7519e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


  Commit: 623225754ba2e6225ea786363d3dd2a41b0c53b9
      https://github.com/dyninst/dyninst/commit/623225754ba2e6225ea786363d3dd2a41b0c53b9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


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

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

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


  Commit: 966d0a45db98deaae714d260c1c5ef58f77306ce
      https://github.com/dyninst/dyninst/commit/966d0a45db98deaae714d260c1c5ef58f77306ce
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


  Commit: 2e99920c9b651d6ea3a4e04b04efea2ea86a6bde
      https://github.com/dyninst/dyninst/commit/2e99920c9b651d6ea3a4e04b04efea2ea86a6bde
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

  Changed paths:
    M dyninstAPI/src/instMapping.h

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


  Commit: ecaa62c5135297278536d6f9ae06b22829d5694e
      https://github.com/dyninst/dyninst/commit/ecaa62c5135297278536d6f9ae06b22829d5694e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 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: 8ede99bc940324e0b22446d87f52a044da7a3f70
      https://github.com/dyninst/dyninst/commit/8ede99bc940324e0b22446d87f52a044da7a3f70
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

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

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

It's always empty.


  Commit: 0568b49b888d912c23f7d4c2f9dca453996070b6
      https://github.com/dyninst/dyninst/commit/0568b49b888d912c23f7d4c2f9dca453996070b6
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

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

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

It's never used.


  Commit: f0047cc163b98e2fbb7ed638fd9a8d295babd9f5
      https://github.com/dyninst/dyninst/commit/f0047cc163b98e2fbb7ed638fd9a8d295babd9f5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 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: 448ad10acbd929c5d05a883fc4099301023ce6e7
      https://github.com/dyninst/dyninst/commit/448ad10acbd929c5d05a883fc4099301023ce6e7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-04-21 (Tue, 21 Apr 2026)

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

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


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

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

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

It's no longer needed.


Compare: https://github.com/dyninst/dyninst/compare/eddc5e864571...b109eba9a8b2

To unsubscribe from these emails, change your notification settings at https://github.com/dyninst/dyninst/settings/notifications