[DynInst_API:] [dyninst/dyninst] c92300: Remove AstNode cost analysis (#2103)


Date: Sun, 18 Jan 2026 15:18:15 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] c92300: Remove AstNode cost analysis (#2103)
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: c923007cdd99e442e8854ec213bf1e4e6b54d7f0
      https://github.com/dyninst/dyninst/commit/c923007cdd99e442e8854ec213bf1e4e6b54d7f0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-01-18 (Sun, 18 Jan 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/freebsd.C
    M dyninstAPI/src/inst-aarch64.C
    M dyninstAPI/src/inst-amdgpu.C
    R dyninstAPI/src/inst-linux.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-winnt.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/inst.C
    M dyninstAPI/src/inst.h

  Log Message:
  -----------
  Remove AstNode cost analysis (#2103)

Cost analysis, introduced by a654415 in 1994, was used to select different AST constructs for collecting performance data during dynamic analysis. It has long been abandoned and the last vestiges were removed by b42dfb6 in 2011.

* Remove AstMiniTrampNode cost analysis

This was only used by AstMiniTrampNode::generateTramp which was removed
by 0b088b5a9.

* Remove initPrimitiveCost

Its usage was removed by b42dfb67 in 2011.

* Remove primitiveCosts

Since initPrimitiveCost is never called, primitiveCosts is always empty.

* CMake formatting



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] c92300: Remove AstNode cost analysis (#2103), Tim Haines <=