[DynInst_API:] [dyninst/dyninst] 340765: Extract, refactor instMapping class (#2218)

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 3407657c5c4090c5f9b56917619ed05163a6538b
      https://github.com/dyninst/dyninst/commit/3407657c5c4090c5f9b56917619ed05163a6538b
  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
    R dyninstAPI/src/inst.C
    M dyninstAPI/src/inst.h
    A dyninstAPI/src/instMapping.h
    M dyninstAPI/src/syscallNotification.C
    M dyninstAPI/src/syscallNotification.h

  Log Message:
  -----------
  Extract, refactor instMapping class (#2218)

* Extract instMapping

* Modernize loops

* Don't typedef enums

* Don't use explicit pointer checks

* Remove dtor

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

* Remove extraneous access specifier

* Reduce string copies

* Remove explicit namespace prefix

* Make is_MTonly const

* Remove unused ctors

* Extract instrumentation location macros

* Remove unused markAs_MTonly

* Use in-class initializers for members not set in ctor

* Remove mt_only

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

* Remove library name

It's always empty.

* Remove 'when'

It's never used.

* Remove 'order'

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

* Inline "fork" ctor

* Remove inst.C

It's no longer needed.

* Move 'args' to init list for "fork" ctor



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