[DynInst_API:] [dyninst/dyninst] 926981: Remove alias of ahptr in parse_frame_one

Mailing List Archives Authenticated access	UW Madison Computer Sciences Department Computer Systems Lab
Date:	Thu, 30 Oct 2025 21:43:41 -0700
From:	wxrdnx <noreply@xxxxxxxxxx>
Subject:	[DynInst_API:] [dyninst/dyninst] 926981: Remove alias of ahptr in parse_frame_one_iteration...
  Branch: refs/heads/angushe/instruction-api-riscv
  Home:   https://github.com/dyninst/dyninst
  Commit: 9269815098ade6eeea930504d907b8893b61204c
      https://github.com/dyninst/dyninst/commit/9269815098ade6eeea930504d907b8893b61204c
  Author: bbiiggppiigg <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M parseAPI/src/Parser.C

  Log Message:
  -----------
  Remove alias of ahptr in parse_frame_one_iteration (#2031)

The alias ah to ahptr is introduced in the commit f9a3e410d1 without apparent reason.


  Commit: 5f28ef2d6a9172b605625bb54ec7a87d16ad874e
      https://github.com/dyninst/dyninst/commit/5f28ef2d6a9172b605625bb54ec7a87d16ad874e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M common/h/Architecture.h

  Log Message:
  -----------
  Add architecture name stringification (#2032)

This is useful when printing error messages for architecture-agnostic code.


  Commit: 0f3f6871cbc9f578f59f5cc638aab140e3025b91
      https://github.com/dyninst/dyninst/commit/0f3f6871cbc9f578f59f5cc638aab140e3025b91
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.C
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.h
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.C
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.h
    M instructionAPI/src/AMDGPU/gfx940/InstructionDecoder-amdgpu-gfx940.C
    M instructionAPI/src/AMDGPU/gfx940/InstructionDecoder-amdgpu-gfx940.h
    M instructionAPI/src/InstructionDecoder-aarch64.C
    M instructionAPI/src/InstructionDecoder-aarch64.h
    M instructionAPI/src/InstructionDecoder-power.C
    M instructionAPI/src/InstructionDecoder-power.h
    M instructionAPI/src/InstructionDecoder-x86.h
    M instructionAPI/src/InstructionDecoderImpl.h

  Log Message:
  -----------
  Remove makeSizeType from InstructionDecoderImpl interface (#2033)

It's only used on x86, so move it there.


  Commit: a3341473c0f3ec217e7372866a040ff5afe04955
      https://github.com/dyninst/dyninst/commit/a3341473c0f3ec217e7372866a040ff5afe04955
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M dwarf/h/dwarfExprParser.h
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/emit-aarch64.h
    M dyninstAPI/src/emit-power.h
    M dyninstAPI/src/emit-x86.C
    M dyninstAPI/src/emit-x86.h
    M dyninstAPI/src/emitter.h
    M dyninstAPI/src/inst-aarch64.C
    M dyninstAPI/src/inst-power.C
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI/src/inst-x86.h
    M dyninstAPI/src/inst.h

  Log Message:
  -----------
  Remove dead code in emitter classes (#2036)

* Remove emitter::emitPIC

It was added by 1bdb443c713 in 2010, but has always been commented-out.

* Remove unused emitter variables

* Remove unused declarations of getInterModule{Var,Func}Addr

These were commented-out by 418b41ea66 in 2019.

* Remove unused PTRACE_REG macros

Added by 18616116a in 2005. Their last usage was removedby cfa91d0260
in 2024.

* Remove virtual register macros

Their usage was removed by 0a3392d0ea3 in 2009.

* Remove GET_GPR macro

Its usage was removed by 4a4b40a242f5 in 2009.

* Remove NUM_FPR_REGISTERS macro

Its usage was removed by bb377beb948 in 2007.

* Remove SAVED_EFLAGS_OFFSET macro

Its usage was removed by 4a4b40a242 in 2009.

* Remove function parameter macros

Their usage was removed by 4a4b40a242 in 2009.

* Remove unsed debug macro

* Remove emitFuncCall

This form was removed by 339cb668 in 2006.

* Remove Register_DWARFtoMachineEnc

It appears its usage was removed by 938db9a0ae in 2007 and
replaced with Register_DWARFtoMachineEnc{32,64}.

* Remove Register_DWARFtoMachineEnc64

Its usage was removed by 9c0aea7f1266d in 2012.

* Remove Register_DWARFtoMachineEnc32

Its usage was removed by 9c0aea7f1266d in 2012.

* Remove EmitterIA32::mt_offset

Its usage was removed by 0a3392d0ea3 in 2009.

* Remove EmitterIA32::mt_offset

Its usage was removed by 0a3392d0ea3 in 2009.

* Remove old comment about PowerPC emitFuncCall

It not longer applies.

* Remove emitAddMem64

Added by 91b6e31e47 in 2006, but never implemented.

* Fix declaration of emitMovImmToRM64

* Fix declaration of emitAddRM64

* Fix declaration of emitMovPCRMToReg64


  Commit: 9e0b0bcbdb9e8e284bfe1a633c67a4dd41d08ab8
      https://github.com/dyninst/dyninst/commit/9e0b0bcbdb9e8e284bfe1a633c67a4dd41d08ab8
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M tests/regression/symtabAPI/CMakeLists.txt
    R tests/regression/symtabAPI/Symtab/CMakeLists.txt
    M tests/regression/symtabAPI/Symtab/file_properties/CMakeLists.txt
    M tests/unit/MachRegister/base_registers/CMakeLists.txt
    M tests/unit/MachRegister/type_queries/CMakeLists.txt
    M tests/unit/common/CMakeLists.txt
    M tests/unit/dataflowAPI/rose/registers/CMakeLists.txt

  Log Message:
  -----------
  Clean up test CMake files (#2038)

* Simplify file name creation for file_properties tests

* Flatten CMakeLists.txt for SymtabAPI

* Remove UNIT_TESTS_INCLUDES

It was removed a long time ago.

* CMake formatting

* Don't reset environment when running file_properties tests


  Commit: b8b5caa1c88b9f86fa9f57d8e6e795e54a8ed368
      https://github.com/dyninst/dyninst/commit/b8b5caa1c88b9f86fa9f57d8e6e795e54a8ed368
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M dataflowAPI/src/AbslocInterface.C
    M dataflowAPI/src/stackanalysis.C
    M dyninstAPI/src/StackMod/StackAccess.C
    M instructionAPI/doc/2-Abstractions.tex
    M instructionAPI/doc/API/BinaryFunction.tex
    M instructionAPI/doc/API/Dereference.tex
    M instructionAPI/doc/API/Expression.tex
    M instructionAPI/doc/API/Immediate.tex
    M instructionAPI/doc/API/MultiRegisterAST.tex
    M instructionAPI/doc/API/RegisterAST.tex
    M instructionAPI/doc/API/TernaryAST.tex
    M instructionAPI/h/BinaryFunction.h
    M instructionAPI/h/Dereference.h
    M instructionAPI/h/Expression.h
    M instructionAPI/h/Immediate.h
    M instructionAPI/h/MultiRegister.h
    M instructionAPI/h/Register.h
    M instructionAPI/h/Ternary.h
    M instructionAPI/src/Immediate.C
    M instructionAPI/src/MultiRegister.C
    M instructionAPI/src/Operand.C
    M instructionAPI/src/Register.C
    M instructionAPI/src/Ternary.C

  Log Message:
  -----------
  Deprecate Expression::getChildren (#2039)

Having in-out parameters makes callsites clunky- particularly for range-
base for loops.


  Commit: 2d046621b8729108b0941a7939afdf268c3292da
      https://github.com/dyninst/dyninst/commit/2d046621b8729108b0941a7939afdf268c3292da
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M instructionAPI/h/Instruction.h
    M instructionAPI/h/InstructionDecoder.h
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.C
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.h
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.C
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.h
    M instructionAPI/src/AMDGPU/gfx940/InstructionDecoder-amdgpu-gfx940.C
    M instructionAPI/src/AMDGPU/gfx940/InstructionDecoder-amdgpu-gfx940.h
    M instructionAPI/src/Instruction.C
    M instructionAPI/src/InstructionDecoder-aarch64.C
    M instructionAPI/src/InstructionDecoder-aarch64.h
    M instructionAPI/src/InstructionDecoder-power.C
    M instructionAPI/src/InstructionDecoder-power.h
    M instructionAPI/src/InstructionDecoder-x86.C
    M instructionAPI/src/InstructionDecoder-x86.h
    M instructionAPI/src/InstructionDecoder.C
    M instructionAPI/src/InstructionDecoderImpl.h

  Log Message:
  -----------
  Always decode instruction operands (#2037)


  Commit: 9b6ec76da59790ffa144b946e19f947dc77a49d0
      https://github.com/dyninst/dyninst/commit/9b6ec76da59790ffa144b946e19f947dc77a49d0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M dataflowAPI/src/RoseInsnFactory.C
    M instructionAPI/src/InstructionDecoderImpl.C
    M instructionAPI/src/InstructionDecoderImpl.h

  Log Message:
  -----------
  Clean up InstructinAPI::InstructionDecoderImpl (#2040)

* Clean up comments

* Remove dead code

* Remove 'MultiRegister' include

* Remove unused Operation_impl include

* Remove redundant 'protected' specifier

* Remove using directive


  Commit: e14278e1b66dfdb7a9230cf4a1d579d74fd4d641
      https://github.com/dyninst/dyninst/commit/e14278e1b66dfdb7a9230cf4a1d579d74fd4d641
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M tests/unit/common/CMakeLists.txt
    A tests/unit/common/registers/CMakeLists.txt
    A tests/unit/common/registers/uniqueness.cpp

  Log Message:
  -----------
  Import register uniqueness tests (#2041)

These were previously in the 'external-tests' repo. They should be
here so they are actually unit tests.


  Commit: 5f3b4dbecbfd25a266bdef8bb2b2032bc5741533
      https://github.com/dyninst/dyninst/commit/5f3b4dbecbfd25a266bdef8bb2b2032bc5741533
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M dataflowAPI/src/AbslocInterface.C
    M instructionAPI/doc/2-Abstractions.tex
    M instructionAPI/doc/API/BinaryFunction.tex
    M instructionAPI/doc/API/Dereference.tex
    M instructionAPI/doc/API/Expression.tex
    M instructionAPI/doc/API/Immediate.tex
    M instructionAPI/doc/API/MultiRegisterAST.tex
    M instructionAPI/doc/API/RegisterAST.tex
    M instructionAPI/doc/API/TernaryAST.tex
    M instructionAPI/h/BinaryFunction.h
    M instructionAPI/h/Dereference.h
    M instructionAPI/h/Expression.h
    M instructionAPI/h/Immediate.h
    M instructionAPI/h/InstructionAST.h
    M instructionAPI/h/MultiRegister.h
    M instructionAPI/h/Register.h
    M instructionAPI/h/Ternary.h
    M instructionAPI/src/Expression.C
    M instructionAPI/src/Immediate.C
    M instructionAPI/src/MultiRegister.C
    M instructionAPI/src/Operand.C
    M instructionAPI/src/Register.C
    M instructionAPI/src/Ternary.C
    M patchAPI/src/PatchBlock.C

  Log Message:
  -----------
  Deprecate Expression::getUses (#2042)

Having in-out parameters makes callsites clunky- particularly for
range-base for loops. The name is also unclear as to what it actually
returns.


  Commit: 72426ea2e0f924d72f34813ffc6f08e1da8e372f
      https://github.com/dyninst/dyninst/commit/72426ea2e0f924d72f34813ffc6f08e1da8e372f
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M tests/unit/CMakeLists.txt
    A tests/unit/dyninstAPI/emitter/CMakeLists.txt
    A tests/unit/dyninstAPI/emitter/emitter_test.h
    A tests/unit/dyninstAPI/emitter/x86_64.cpp

  Log Message:
  -----------
  Add tests for x86_64 emitter (#2017)


  Commit: 2b092a7fdae9856810c133e93be1a38245c33bb0
      https://github.com/dyninst/dyninst/commit/2b092a7fdae9856810c133e93be1a38245c33bb0
  Author: wxrdnx <67510189+wxrdnx@xxxxxxxxxxxxxxxxxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M cmake/DyninstPlatform.cmake
    M cmake/tpls/DyninstElfUtils.cmake
    M parseAPI/h/CodeSource.h
    M parseAPI/src/SymtabCodeSource.C
    M symtabAPI/CMakeLists.txt
    M symtabAPI/h/relocationEntry.h
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/Object-elf.h
    M symtabAPI/src/Object.h
    M symtabAPI/src/emitElf.C
    A symtabAPI/src/emitElfStatic-riscv64.C
    M symtabAPI/src/emitElfStatic.C
    A symtabAPI/src/relocationEntry-elf-riscv64.C

  Log Message:
  -----------
  Add RISC-V Specific Elf Section Parsing (#1966)


  Commit: d7fd45db91469e353593f47ecfd7bb590417d392
      https://github.com/dyninst/dyninst/commit/d7fd45db91469e353593f47ecfd7bb590417d392
  Author: wxrdnx <wxrdnx@xxxxxxxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M instructionAPI/src/InstructionDecoderImpl.C

  Log Message:
  -----------
  Add RISC-V Instruction API


  Commit: 588f48692efd8b3406592a7b827a1f037edc4934
      https://github.com/dyninst/dyninst/commit/588f48692efd8b3406592a7b827a1f037edc4934
  Author: wxrdnx <wxrdnx@xxxxxxxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    M .github/workflows/dependency-version.yaml
    M CMakeLists.txt
    A cmake/tpls/DyninstCapstone.cmake
    M common/CMakeLists.txt
    A common/h/registers/register_set.h
    M docker/Dockerfile.fedora
    M docker/Dockerfile.ubuntu
    A docker/build_capstone.sh
    M docker/dependencies.versions
    M instructionAPI/CMakeLists.txt
    M instructionAPI/h/Instruction.h
    M instructionAPI/src/Instruction.C
    M instructionAPI/src/InstructionDecoderImpl.C
    A instructionAPI/src/riscv/categories.h
    A instructionAPI/src/riscv/decoder.C
    A instructionAPI/src/riscv/decoder.h
    A instructionAPI/src/riscv/opcode_xlat.C
    A instructionAPI/src/riscv/opcode_xlat.h
    A instructionAPI/src/riscv/register_xlat.C
    A instructionAPI/src/riscv/register_xlat.h
    A instructionAPI/src/riscv/type_conversion.h

  Log Message:
  -----------
  Update Capstone


  Commit: c554cb419ced055fd2ae9e50f25c990ab8a4e122
      https://github.com/dyninst/dyninst/commit/c554cb419ced055fd2ae9e50f25c990ab8a4e122
  Author: wxrdnx <wxrdnx@xxxxxxxxxxxxxx>
  Date:   2025-10-30 (Thu, 30 Oct 2025)

  Changed paths:
    A common/src/arch-riscv64.h
    M instructionAPI/CMakeLists.txt
    M instructionAPI/src/Instruction.C
    M instructionAPI/src/riscv/categories.h
    M instructionAPI/src/riscv/decoder.C
    M instructionAPI/src/riscv/decoder.h

  Log Message:
  -----------
  Update pseudo instructions (wip)


Compare: https://github.com/dyninst/dyninst/compare/8866f721eb6a...c554cb419ced

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] 926981: Remove alias of ahptr in parse_frame_one_iteration..., wxrdnx <=
Previous by Date:	[DynInst_API:] [dyninst/dyninst] 159f70: CMake formatting, Tim Haines
Next by Date:	[DynInst_API:] [dyninst/dyninst] 29499c: Fix typos in Dyninst's kernel descriptor code, Ronak Chauhan
Previous by Thread:	[DynInst_API:] [dyninst/dyninst] 91ff57: Fix lui signedness problem, wxrdnx
Next by Thread:	[DynInst_API:] [dyninst/dyninst] 92ab20: Change zero flag comment for consistency, wxrdnx
Indexes:	[Date] [Thread]
Mailing List Archives

Authenticated access

[DynInst_API:] [dyninst/dyninst] 926981: Remove alias of ahptr in parse_frame_one_iteration...
