Branch: refs/heads/thaines/cleanup_AstNode
  Home:   https://github.com/dyninst/dyninst
  Commit: 526f044312ab06bd0d52d5cf8a64f6d80996ee90
      
https://github.com/dyninst/dyninst/commit/526f044312ab06bd0d52d5cf8a64f6d80996ee90
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_function.C
    M dyninstAPI/src/BPatch_snippet.C
    M dyninstAPI/src/IAPI_to_AST.C
    A dyninstAPI/src/OperandType.h
    M dyninstAPI/src/addressSpace.C
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/dynProcess.C
    M dyninstAPI/src/inst-aarch64.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/linux-x86.C
    M dyninstAPI/src/syscallNotification.C

  Log Message:
  -----------
  Extract operandType

This centralizes the type and its functions and will help with
possibly refactoring the operand-specific methods out of AstNode
some day.


  Commit: cba16f78d18fdb969d849cb6866b0482544561ae
      
https://github.com/dyninst/dyninst/commit/cba16f78d18fdb969d849cb6866b0482544561ae
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/OperandType.h
    M dyninstAPI/src/ast.C

  Log Message:
  -----------
  Remove operandType::DataReg

Its only usage was in BPatch_regExpr that was was deprecated by 4d15de8
in 2008 and removed by f4ee3410b in 2020.


  Commit: 2f2fb7da188719057dae01c0857f40daa8fb0a5a
      
https://github.com/dyninst/dyninst/commit/2f2fb7da188719057dae01c0857f40daa8fb0a5a
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.C

  Log Message:
  -----------
  Use simpler, faster isPowerOf2 in ast.C


  Commit: 862fe3457eb58271a9bb5daee5572245f707bad4
      
https://github.com/dyninst/dyninst/commit/862fe3457eb58271a9bb5daee5572245f707bad4
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/inst.h

  Log Message:
  -----------
  Move isPowerOf2(int,int&) into inst.h

This is the only place it's used now.


  Commit: 359514592f70dd652221dccf26b4c970f0bc3fe7
      
https://github.com/dyninst/dyninst/commit/359514592f70dd652221dccf26b4c970f0bc3fe7
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Extract AstNode::format(opCode) into opcode.h


  Commit: 957874c5325da2389de91bfa50ec059d0d75ee34
      
https://github.com/dyninst/dyninst/commit/957874c5325da2389de91bfa50ec059d0d75ee34
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/inst.h

  Log Message:
  -----------
  Remove unneeded include of 'opcode.h'


  Commit: 0efbffed0af259891d7211983190870d5ac7836b
      
https://github.com/dyninst/dyninst/commit/0efbffed0af259891d7211983190870d5ac7836b
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/registerSpace.h

  Log Message:
  -----------
  Move declaration of emitLoadPreviousStackFrameRegister to registerSpace

This is the only place it's used.


  Commit: 3c224eaf244e95a5a9f88726c060465c0d6ccf7b
      
https://github.com/dyninst/dyninst/commit/3c224eaf244e95a5a9f88726c060465c0d6ccf7b
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.h

  Log Message:
  -----------
  Remove SCAST_AST macro

It's no longer used.


  Commit: bb41e0c73c52188dc8c66d9e4e685ae1b9f430a3
      
https://github.com/dyninst/dyninst/commit/bb41e0c73c52188dc8c66d9e4e685ae1b9f430a3
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/baseTramp.C

  Log Message:
  -----------
  Move DCAST_AST to baseTramp.C

That's the only place it's used.


  Commit: 21e66b0750fe224076f431d76a37d56fee21bb1e
      
https://github.com/dyninst/dyninst/commit/21e66b0750fe224076f431d76a37d56fee21bb1e
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.h

  Log Message:
  -----------
  Clean up comments


  Commit: 5408bc6949dd306a12cd0efc979d6266b3af83d9
      
https://github.com/dyninst/dyninst/commit/5408bc6949dd306a12cd0efc979d6266b3af83d9
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

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

  Log Message:
  -----------
  Clean up includes


  Commit: ec35b5712f4f8aba1b09bf4eee66d1ed0f6dfb30
      
https://github.com/dyninst/dyninst/commit/ec35b5712f4f8aba1b09bf4eee66d1ed0f6dfb30
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.h

  Log Message:
  -----------
  Update include guard name in ast.h


  Commit: 5457c7b6de85ac9f01fb635c5fa3efd1bacf7589
      
https://github.com/dyninst/dyninst/commit/5457c7b6de85ac9f01fb635c5fa3efd1bacf7589
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/emit-x86.C
    M dyninstAPI/src/inst-x86.C

  Log Message:
  -----------
  Remove reference-counting in AstNode

1. referenceCount is never directly checked, only modified
2. decRefCount _always_ returns true
3. Callsites for decRefCount only check if it returns true

These combined mean that modifying referenceCount has no observable
side effect.


  Commit: 6747cd64119f753b8089d45dc98d6f23c0ccdad1
      
https://github.com/dyninst/dyninst/commit/6747cd64119f753b8089d45dc98d6f23c0ccdad1
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Remove unused 'breakOp' operator type

It was added by 0a3392d0e in 2009, but never implemented.


  Commit: a878af87120fa6f95b8d19c3ecb34d7fb5464bb3
      
https://github.com/dyninst/dyninst/commit/a878af87120fa6f95b8d19c3ecb34d7fb5464bb3
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Remove unused 'doOp' operatory type

It was added by ccc39c2b2 in 1998, but never implemented.


  Commit: 61ad7ecd90bd3a9a899cfa70fa2815638d6e2b36
      
https://github.com/dyninst/dyninst/commit/61ad7ecd90bd3a9a899cfa70fa2815638d6e2b36
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Remove unused 'funcCallOp' operator type

It was added by 6616f912a in 2009 and removed by 73039f3d931 in 2011.


  Commit: 09c5cd8fd175eba4b126755e4e25c9694d2aacbb
      
https://github.com/dyninst/dyninst/commit/09c5cd8fd175eba4b126755e4e25c9694d2aacbb
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

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

  Log Message:
  -----------
  Remove unused 'getSysParamOp' operator type

It was added by 0aa3b67be in 1996 for SunOS and
the last vestiges were removed by dee8900 in
2011.


  Commit: 44ab11b457d5fc9a87e58283c0607be7cc1a6162
      
https://github.com/dyninst/dyninst/commit/44ab11b457d5fc9a87e58283c0607be7cc1a6162
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

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

  Log Message:
  -----------
  Remove unused 'getSysRetValOp' operator type

It was added by d2d5831e7 in 1996 for SunOS and
the last vestiges were removed by dee8900 in
2011.


  Commit: 65aebe4ecc6f232ace42ee52fe64337123e08502
      
https://github.com/dyninst/dyninst/commit/65aebe4ecc6f232ace42ee52fe64337123e08502
  Author: Tim Haines <[email protected]>
  Date:   2026-01-24 (Sat, 24 Jan 2026)

  Changed paths:
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Remove unused 'loadStateOp','storeStateOp' operator types

They were added by 18ab46621 in 2006 but never implemented.


  Commit: 069ac6d7b1fb42e66c484e76e6c7102f9e98992b
      
https://github.com/dyninst/dyninst/commit/069ac6d7b1fb42e66c484e76e6c7102f9e98992b
  Author: Tim Haines <[email protected]>
  Date:   2026-01-25 (Sun, 25 Jan 2026)

  Changed paths:
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Remove unused 'trampPreamble' operator type

Its last usage as an operation was removed by 123a8c7 in 2006.


  Commit: dd80953a3ab6e701f52ca90038fdc0849270511f
      
https://github.com/dyninst/dyninst/commit/dd80953a3ab6e701f52ca90038fdc0849270511f
  Author: Tim Haines <[email protected]>
  Date:   2026-01-25 (Sun, 25 Jan 2026)

  Changed paths:
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/opcode.h

  Log Message:
  -----------
  Remove unused 'updateCostOp' operator type

Its last usage as an operation was removed by cb0b86cc in 2006.


Compare: https://github.com/dyninst/dyninst/compare/05576637ef05...dd80953a3ab6

To unsubscribe from these emails, change your notification settings at 
https://github.com/dyninst/dyninst/settings/notifications
_______________________________________________
Dyninst-api mailing list
[email protected]
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api

Reply via email to