Branch: refs/heads/thaines/cleanup_AstNode
Home: https://github.com/dyninst/dyninst
Commit: 608eb70fa2e04d50028a724eb9cc0d27d7471a83
https://github.com/dyninst/dyninst/commit/608eb70fa2e04d50028a724eb9cc0d27d7471a83
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove deepCopy
It was added by 21d1427b9 in 2010, but never used.
Commit: aeb602012ac80de8970d6e9f5bb0141793c6efaf
https://github.com/dyninst/dyninst/commit/aeb602012ac80de8970d6e9f5bb0141793c6efaf
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove Dyn_C error reporting variables
These were only used in the deepCopy member functions.
Commit: f310c0f179dcb026b712ccf9e40126f9de467b38
https://github.com/dyninst/dyninst/commit/f310c0f179dcb026b712ccf9e40126f9de467b38
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove nodeType
Its usage was removed by 20116a1026 in 2006
Commit: 937a8c41c5d59c503fa3058278b737d7f58e0d1d
https://github.com/dyninst/dyninst/commit/937a8c41c5d59c503fa3058278b737d7f58e0d1d
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove cleanRegTracker
This was added by 123a8c7f31 in 2010, but never implemented.
Commit: 5833968512c711c08a2eebc223725c125cc8a3d5
https://github.com/dyninst/dyninst/commit/5833968512c711c08a2eebc223725c125cc8a3d5
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
M dyninstAPI/src/baseTramp.h
Log Message:
-----------
Move cfjRet_t into baseTramp
This is the only place it's used.
Commit: 021192a9417c4498c90d3a751ba899f031d66a52
https://github.com/dyninst/dyninst/commit/021192a9417c4498c90d3a751ba899f031d66a52
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove dataReqNode declaration
It's never used.
Commit: b8005f75b938f56be60871056d1769f363331379
https://github.com/dyninst/dyninst/commit/b8005f75b938f56be60871056d1769f363331379
Author: Tim Haines <[email protected]>
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/codegen.h
A dyninstAPI/src/regTracker.C
A dyninstAPI/src/regTracker.h
Log Message:
-----------
Extract regTracker_t
I'm suspicious of the gymanastics in AstNode::generateCode, but
I'll leave it as-is for now.
Commit: 72e7b4459e0fb8d2115eca23e0506916807e3a2e
https://github.com/dyninst/dyninst/commit/72e7b4459e0fb8d2115eca23e0506916807e3a2e
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove 'void' parameter
This isn't legal c++.
Commit: c2ea5a83dc59669bdbdb4ed466fe32a56414d598
https://github.com/dyninst/dyninst/commit/c2ea5a83dc59669bdbdb4ed466fe32a56414d598
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove checkUseCount
Its usage was removed by 123a8c7f in 2006.
Commit: 60b9b99a8e0deee24c4f3df1eb65b35306fdfc3c
https://github.com/dyninst/dyninst/commit/60b9b99a8e0deee24c4f3df1eb65b35306fdfc3c
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove printUserCount
Its usage was removed by 123a8c7f3 in 2006.
Commit: 199e9f84237cdf232516e6e2cf6148277aefb96d
https://github.com/dyninst/dyninst/commit/199e9f84237cdf232516e6e2cf6148277aefb96d
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove AstNode::getArgs
It was added by 21d1427b9 in 2010, but never used.
Commit: 36520f243ab5b33de9d29cd010642077c49f29e2
https://github.com/dyninst/dyninst/commit/36520f243ab5b33de9d29cd010642077c49f29e2
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove setChildren
It was added by 21d1427b9 in 2010, but never used.
Commit: 1c6521886b0e29261133aaf99ed0a0ad2d0cd245
https://github.com/dyninst/dyninst/commit/1c6521886b0e29261133aaf99ed0a0ad2d0cd245
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Clean up canBeKept
Commit: c05a0bad80f679e1df415954c308bd564ae3a83d
https://github.com/dyninst/dyninst/commit/c05a0bad80f679e1df415954c308bd564ae3a83d
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove stealRegister
It was added by 0380c4db951 in 2006, but never implemented.
Commit: a40d6322f4eeb24fd5d6856a59686b669de51ab9
https://github.com/dyninst/dyninst/commit/a40d6322f4eeb24fd5d6856a59686b669de51ab9
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove subpath
Its usage and definition were removed by 123a8c7f31 in 2006.
Commit: a55d11251ba9ee4b0197c8cb19a3c9f62cb4245d
https://github.com/dyninst/dyninst/commit/a55d11251ba9ee4b0197c8cb19a3c9f62cb4245d
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Move children into AstNode
This simplifies the implementations.
Commit: fc65814916155c50e001871a9a8c0502f8102aa7
https://github.com/dyninst/dyninst/commit/fc65814916155c50e001871a9a8c0502f8102aa7
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove AstNode(AstNodePtr)
Its original implementation 'AstNode(AstNode *src)' was removed by
5dabf3ca8b1 in 2006. The interface was changed to its current form by
e7c183b934 in 2007, but was never re-implemented.
Commit: 522a665dedad113c46194b1dcb6392db9071c158
https://github.com/dyninst/dyninst/commit/522a665dedad113c46194b1dcb6392db9071c158
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove AstNode::getTreeSize
It was added by 97dc86ee3f0 in 2006, but never used.
Commit: becd700fc295d18ef35c22161722e91277c9dc81
https://github.com/dyninst/dyninst/commit/becd700fc295d18ef35c22161722e91277c9dc81
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Remove accessesParam
Its usage was removed by 47c4dfe96 in 2010.
Commit: b56c9b85974dbb3abc5023766f75f8420d387d5e
https://github.com/dyninst/dyninst/commit/b56c9b85974dbb3abc5023766f75f8420d387d5e
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Provide default for containsFuncCall
Commit: e7fb96dd5d91f951682d2fdcba7354ae81e0821f
https://github.com/dyninst/dyninst/commit/e7fb96dd5d91f951682d2fdcba7354ae81e0821f
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Provide a default for setVariableAST
Commit: c3f0a7ea0d8e3d23f60657d7d3d298a156b215ad
https://github.com/dyninst/dyninst/commit/c3f0a7ea0d8e3d23f60657d7d3d298a156b215ad
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Provide a default for usesAppRegister
Commit: 6609ba17b8ff51d9fd026f85e047dac3995933d4
https://github.com/dyninst/dyninst/commit/6609ba17b8ff51d9fd026f85e047dac3995933d4
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Require generateCode_phase2
All of the ASTs already implement it. This also converts the runtime check
into a compile-time one.
Commit: 555516c495338c50f57972426f460ffba1374863
https://github.com/dyninst/dyninst/commit/555516c495338c50f57972426f460ffba1374863
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/CMakeLists.txt
A dyninstAPI/src/AstNode.C
A dyninstAPI/src/AstNode.h
M dyninstAPI/src/ast.C
M dyninstAPI/src/ast.h
Log Message:
-----------
Extract NodeAst
Commit: c195819f892a449217e0ff72cdfa3ea32820a181
https://github.com/dyninst/dyninst/commit/c195819f892a449217e0ff72cdfa3ea32820a181
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/AstNode.C
M dyninstAPI/src/AstNode.h
Log Message:
-----------
AstNode: use in-class member var initializers
Commit: 67219193bae9129696a22c6f26b6ce6abbf62807
https://github.com/dyninst/dyninst/commit/67219193bae9129696a22c6f26b6ce6abbf62807
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/AstNode.h
Log Message:
-----------
AstNode: remove labelNode(std::string)
This should have been removed by eac8cf2c8.
Commit: 2e618fd6efbe63a4e16a268f6a8d11f471eea673
https://github.com/dyninst/dyninst/commit/2e618fd6efbe63a4e16a268f6a8d11f471eea673
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/AstNode.C
M dyninstAPI/src/AstNode.h
Log Message:
-----------
AstNode: use default dtor
Commit: 4fadaede9d49a2d5b2330cf92265c9d7788ec9d2
https://github.com/dyninst/dyninst/commit/4fadaede9d49a2d5b2330cf92265c9d7788ec9d2
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/AstNode.C
M dyninstAPI/src/AstNode.h
Log Message:
-----------
AstNode: inline getType
Commit: 79332214036b112b1689f33c2aa8c2aad225e6d5
https://github.com/dyninst/dyninst/commit/79332214036b112b1689f33c2aa8c2aad225e6d5
Author: Tim Haines <[email protected]>
Date: 2026-01-18 (Sun, 18 Jan 2026)
Changed paths:
M dyninstAPI/src/AstNode.C
M dyninstAPI/src/AstNode.h
Log Message:
-----------
NodeAst: whitespace, clean up comments
Compare:
https://github.com/dyninst/dyninst/compare/608eb70fa2e0%5E...79332214036b
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
