[DynInst_API:] [dyninst/dyninst] 0f7866: [dyninstAPI] Extract, refactor parsing classes (#2...


Date: Tue, 23 Jun 2026 14:07:53 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 0f7866: [dyninstAPI] Extract, refactor parsing classes (#2...
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 0f7866ef361239810051739509690d9616e5b14b
      https://github.com/dyninst/dyninst/commit/0f7866ef361239810051739509690d9616e5b14b
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-06-23 (Tue, 23 Jun 2026)

  Changed paths:
    M dyninstAPI/src/BPatch/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch/BPatch_edge.C
    M dyninstAPI/src/hybridCallbacks.C
    M dyninstAPI/src/image.C
    M dyninstAPI/src/image.h
    M dyninstAPI/src/mapped_object.C
    M dyninstAPI/src/mapped_object.h
    M dyninstAPI/src/parsing/CMakeLists.txt
    A dyninstAPI/src/parsing/DynCFGFactory.C
    A dyninstAPI/src/parsing/DynCFGFactory.h
    A dyninstAPI/src/parsing/DynParseCallback.C
    A dyninstAPI/src/parsing/DynParseCallback.h
    R dyninstAPI/src/parsing/Parsing-arch.C
    R dyninstAPI/src/parsing/Parsing.C
    R dyninstAPI/src/parsing/Parsing.h
    M dyninstAPI/src/parsing/parse_func.C
    M dyninstAPI/src/parsing/parse_func.h
    M dyninstAPI/src/patching/function.C

  Log Message:
  -----------
  [dyninstAPI] Extract, refactor parsing classes (#2289)

* Extract DynCFGFactory

* DynCFGFactory: Remove dead code

* DynCFGFactory: Remove allocation stats

These are unused debugging checks.

* DynCFGFactory: Clean up mkblock

* DynCFGFactory: mkfunc: use boost::lock_guard

* DynCFGFactory: mkfunc: simplify 'st' initialization

* Use default-init for 'stf'

* DynCFGFactory: mkfunc: simplify pdmod calculation

* DynCFGFactory: mkfunc: simplify return flow

* DynCFGFactory: Inline ctor

* DynCFGFactory: explicitly default dtor

This still suppresses compiler declarations of copy/move operations.

* DynCFGFactory: mksink cleanup

* Extract DynParseCallback

This also inlines codeBytesUpdateCB.

* Remove Parsing.{h,C}

They aren't needed anymore.

* Extract DynParseCallback::instruction_cb

There's no need to have a compile guard for it.

* Use instruction_cb on aarch64

This doesn't actually do anything since 'isReturnAddrSave' is always false for
aarch64, but it makes it consistent with the previous compile guards.



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] 0f7866: [dyninstAPI] Extract, refactor parsing classes (#2..., Tim Haines <=