[DynInst_API:] [dyninst/dyninst] 493ace: Remove unused pare_func_registers::specialPurposeR...

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/thaines/split_AstNode_classes
  Home:   https://github.com/dyninst/dyninst
  Commit: 493ace92025837d5e2468ddc8e75430df6f7800f
      https://github.com/dyninst/dyninst/commit/493ace92025837d5e2468ddc8e75430df6f7800f
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-02 (Mon, 02 Feb 2026)

  Changed paths:
    M dyninstAPI/src/parse-cfg.h

  Log Message:
  -----------
  Remove unused pare_func_registers::specialPurposeRegisters (#2136)

It was added by 7c2938c70 in 2006, but never used.


  Commit: d7942b68f59058941de6d86dc929f3f1171aef4e
      https://github.com/dyninst/dyninst/commit/d7942b68f59058941de6d86dc929f3f1171aef4e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-03 (Tue, 03 Feb 2026)

  Changed paths:
    M .github/workflows/binary-tests.yaml
    M .github/workflows/build-opts.yaml
    M .github/workflows/cmake-formatting.yaml
    M .github/workflows/codegen-cross-build.yaml
    M .github/workflows/compiler-multibuild.yaml
    M .github/workflows/consumers.yaml
    M .github/workflows/dependency-version.yaml
    M .github/workflows/dev-containers.yaml
    M .github/workflows/libabigail.yaml
    M .github/workflows/pr-tests.yaml
    M .github/workflows/purge-workflows.yaml
    M .github/workflows/refresh-containers.yaml
    M .github/workflows/release-containers.yaml
    M .github/workflows/spack-build.yaml
    M .github/workflows/system-libs.yaml

  Log Message:
  -----------
  GitHub CI: auto-cancel in-progress workflows (#2139)

Pushing changes to a PR's branch will relaunch all of the workflows. This
change ensures that any workflows running against that PR are terminated
to free resources and run with the updated branch.


  Commit: 0cc8dc400337c8f8e549d363259952130d71b8c2
      https://github.com/dyninst/dyninst/commit/0cc8dc400337c8f8e549d363259952130d71b8c2
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-03 (Tue, 03 Feb 2026)

  Changed paths:
    M proccontrol/h/PCProcess.h
    M proccontrol/src/int_process.h
    M proccontrol/src/process.C

  Log Message:
  -----------
  Add a synchronized breakpoint type (#2137)

This can be used to ensure that all threads in a process are stopped
before the breakpoint is executed.


  Commit: 8926e8e3b30aadaab90eea1bbea03024c7f7266c
      https://github.com/dyninst/dyninst/commit/8926e8e3b30aadaab90eea1bbea03024c7f7266c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-04 (Wed, 04 Feb 2026)

  Changed paths:
    M common/src/dyn_register.h

  Log Message:
  -----------
  Make Dyninst::Register constructors noexcept (#2140)

For std::vector<T>::emplace_back, the standard notes

  If the move constructor of T is not noexcept and is not CopyInsertable
  into *this, vector will use the throwing move constructor.

gcc-15 and newer warn via '-Werror=noexcept' when the move_if_noexcept
condition fails and recommends making the ctor noexcept. This is not a
requirement, but the CI jobs build with -Werror.


  Commit: 461361dac57cee73d9ba6c26d7c352cc5f81681a
      https://github.com/dyninst/dyninst/commit/461361dac57cee73d9ba6c26d7c352cc5f81681a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-04 (Wed, 04 Feb 2026)

  Changed paths:
    M cmake/DyninstOptimization.cmake

  Log Message:
  -----------
  Add clang flag to split hot/cold code (#2138)


  Commit: eb92b3449e6fc705341b3f3a59961e31163391be
      https://github.com/dyninst/dyninst/commit/eb92b3449e6fc705341b3f3a59961e31163391be
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-05 (Thu, 05 Feb 2026)

  Changed paths:
    M dyninstAPI/src/parse-cfg.h

  Log Message:
  -----------
  Remove parse_func::writesSPRs (#2141)

It was added by ad610d9a0c90 in 2006, but never implemented.


  Commit: df5ffba700fe21ce3139ba822f8c5baab4c8ae97
      https://github.com/dyninst/dyninst/commit/df5ffba700fe21ce3139ba822f8c5baab4c8ae97
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-05 (Thu, 05 Feb 2026)

  Changed paths:
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/pcEventHandler.C
    M proccontrol/src/event.C
    M proccontrol/src/handler.C

  Log Message:
  -----------
  Add proccontrol support for threads spawned before reaching main (#2131)

* Use a synchronized breakpoint for 'main'

If a thread is spawned before reaching main (e.g., in a .ctor entry),
proccontrol does not observe events from it and so assumes it is safe
to remove the breakpoint at 'main'. It is unclear at this time why no
events are observed. Particularly becuase there is a check that _should_
catch this case:

  BPatch::processCreate
          V
  BPatch_process::BPatch_process
          V
  PCProcess::createProcess
          V
  Process::createProcess
          V
  int_process::createProcess
          V
  PCProcess::bootstrapProcess()
          V
  assert( pcProc_->allThreadsStopped() );

co-authored by Kian Cossettini @ AMD

* Ignore events for destroyed threads launched before main

If a thread is launched and destroyed before main is reached, there
could be events in the queue that belong to that dead thread. We should
ignore them. This seems at odds with proccontrol's failure to detect
that there are threads alive before main is reached, but there is a
user-reported crash that is fixed by these changes.

Co-authored-by: Kian Cossettini <Kian.Cossettini@xxxxxxx>


  Commit: 2b27c01a702f02dd5b4c6482e3f0fe49c2c09f2a
      https://github.com/dyninst/dyninst/commit/2b27c01a702f02dd5b4c6482e3f0fe49c2c09f2a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-07 (Sat, 07 Feb 2026)

  Changed paths:
    M .github/workflows/dev-containers.yaml
    M .github/workflows/purge-workflows.yaml
    M .github/workflows/refresh-containers.yaml
    M .github/workflows/release-containers.yaml

  Log Message:
  -----------
  GitHub CI: don't autocancel container workflows (#2147)

'Refresh base and dev containers' depends on
'Build and Deploy Development Containers'. 'Purge old workflow runs'
isn't in that dependency chain, but it could be added to a longer
workflow in the future. It doesn't matter if it is run multiple times.


  Commit: d82afffc64f3403e7ed6055e170286fc3560a865
      https://github.com/dyninst/dyninst/commit/d82afffc64f3403e7ed6055e170286fc3560a865
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-07 (Sat, 07 Feb 2026)

  Changed paths:
    M dyninstAPI/src/parse-cfg.C

  Log Message:
  -----------
  Fix return value in parse_func::IsLeafFunc (#2145)

A leaf function is one that has _no_ call edges.


  Commit: 8493016bae509d3a73f6747dafe33c7af6013e14
      https://github.com/dyninst/dyninst/commit/8493016bae509d3a73f6747dafe33c7af6013e14
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-10 (Tue, 10 Feb 2026)

  Changed paths:
    M dyninstAPI/h/BPatch_function.h
    M dyninstAPI/src/BPatch_function.C
    M dyninstAPI/src/function.h
    M dyninstAPI/src/parse-cfg.C
    M dyninstAPI/src/parse-cfg.h

  Log Message:
  -----------
  Remove unused code in parse_func, parse_block (#2142)

* Remove parse_func::containsSPRWrites_

It was added by ad610d9a0 in 2006, but never used.

* Remove parse_func::containsSharedBlocks_

Its usage was removed by 62b7d1f21 in 2010.

This also removes the following functions that directly or indirectly
depend on it:

  parse_func::containsSharedBlocks()
  func_instance::containsSharedBlocks()
  BPatch_function::containsSharedBlocks()

The one in BPatch_function was explicitly marked as not to be
documented, so it's assumed users never use it.

* Remove parse_func::invalidateLiveness_

Its usage was removed by 70328949c in 2011.

* Remove parse_func::o7_live

Its usage was removed by dee8900a in 2011.

* Remove parse_func::calcBlockLevelLiveness

Its usage was removed by 799b99dc2 in 2013.

* Remove parse_block::getLivenessIn

Its usage was removed by 18a7ca0a4 in 2011

* Remove parse_block::blockSet

Its usage was removed by 18a7ca0a4 in 2011.

* Remove parse_block::getLivenessOut

Its usage was removed by 18a7ca0a4 in 2011.

* Remove redundant typedef for Insns

It's defined in ParseAPI::Block.

* Use default dtor for parse_block


  Commit: 081f5af2d3bb649ba16943e40301526c7c1e3738
      https://github.com/dyninst/dyninst/commit/081f5af2d3bb649ba16943e40301526c7c1e3738
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-11 (Wed, 11 Feb 2026)

  Changed paths:
    M dyninstAPI/src/parse-aarch64.C
    M dyninstAPI/src/parse-amdgpu.C
    M dyninstAPI/src/parse-cfg.C
    M dyninstAPI/src/parse-cfg.h
    M dyninstAPI/src/parse-power.C

  Log Message:
  -----------
  Remove unused OpenMP parsing in parse_func (#2135)

* Remove parse_func::parseOMPParent

Functionality removed by dee8900ad35 in 2011. aarch64 was added by
311466419b in 2015, but never implemented.

* Remove parse_func::calcParentFunc

Its last usage was removed by b7899c9de7c37 in 2014.

* Remove parse_func::OMPparsed

It's not used anywhere.

* Remove parse_func::OMPparsed_

It is only ever checked/updated in parseOMPFunc on ppc, so has no effect.

* Remove parse_func::parseOMPFunc

Its no longer used.

* Remove parse_func::addRegion

ITs no longer used.

* Remove parse_func::parseOMP

It's last usage was removed by b7899c9de in 2014.


  Commit: 551b6dc41c46fa830c263c15cf586c520fe5d1a8
      https://github.com/dyninst/dyninst/commit/551b6dc41c46fa830c263c15cf586c520fe5d1a8
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-13 (Fri, 13 Feb 2026)

  Changed paths:
    M dyninstAPI/src/mapped_object.C
    M dyninstAPI/src/mapped_object.h
    M dyninstAPI/src/parse-aarch64.C
    M dyninstAPI/src/parse-amdgpu.C
    M dyninstAPI/src/parse-power.C
    M dyninstAPI/src/parse-x86.C

  Log Message:
  -----------
  Consolidate mapped_object ctor/dtor lookup (#2134)

* Merge mapped_object ctor/dtor lookup

All of the architectures do the same thing, so there's no need for
separate implementations. There's no need to assert for unimplemented
architectures.

* Merge the implementations into findGlobalFunc


  Commit: 7619154eec78d8f0bafb00be0d6a512339ab40cf
      https://github.com/dyninst/dyninst/commit/7619154eec78d8f0bafb00be0d6a512339ab40cf
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-16 (Mon, 16 Feb 2026)

  Changed paths:
    M .github/workflows/consumers.yaml

  Log Message:
  -----------
  GitHub CI: Remove STAT patch (#2153)

It has been merged on their side.


  Commit: 8788c179e07e9970fff4fe358a98c5b0433999c1
      https://github.com/dyninst/dyninst/commit/8788c179e07e9970fff4fe358a98c5b0433999c1
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-16 (Mon, 16 Feb 2026)

  Changed paths:
    M dyninstAPI/src/mapped_object.C

  Log Message:
  -----------
  Remove shadowed variable in mapped_object::findGlobalFunc (#2154)

This was reported by gcc-7 as shadowing the inherited member function PatchObject::funcs(Iter).


  Commit: b3dfc40a6e64a8f28436a603666202e64fc34db2
      https://github.com/dyninst/dyninst/commit/b3dfc40a6e64a8f28436a603666202e64fc34db2
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-02-16 (Mon, 16 Feb 2026)

  Changed paths:
    M .github/workflows/binary-tests.yaml
    M .github/workflows/build-opts.yaml
    M .github/workflows/cmake-formatting.yaml
    M .github/workflows/codegen-cross-build.yaml
    M .github/workflows/compiler-multibuild.yaml
    M .github/workflows/consumers.yaml
    M .github/workflows/dependency-version.yaml
    M .github/workflows/libabigail.yaml
    M .github/workflows/pr-tests.yaml
    M .github/workflows/spack-build.yaml
    M .github/workflows/system-libs.yaml
    M cmake/DyninstOptimization.cmake
    M common/src/dyn_register.h
    M dyninstAPI/h/BPatch_function.h
    M dyninstAPI/src/BPatch_function.C
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/function.h
    M dyninstAPI/src/mapped_object.C
    M dyninstAPI/src/mapped_object.h
    M dyninstAPI/src/parse-aarch64.C
    M dyninstAPI/src/parse-amdgpu.C
    M dyninstAPI/src/parse-cfg.C
    M dyninstAPI/src/parse-cfg.h
    M dyninstAPI/src/parse-power.C
    M dyninstAPI/src/parse-x86.C
    M dyninstAPI/src/pcEventHandler.C
    M proccontrol/h/PCProcess.h
    M proccontrol/src/event.C
    M proccontrol/src/handler.C
    M proccontrol/src/int_process.h
    M proccontrol/src/process.C

  Log Message:
  -----------
  Merge branch 'master' into thaines/split_AstNode_classes


Compare: https://github.com/dyninst/dyninst/compare/491827b7db47...b3dfc40a6e64

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