[DynInst_API:] [dyninst/dyninst] eda0eb: Deprecate Expression::getUses


Date: Sun, 26 Oct 2025 11:48:06 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] eda0eb: Deprecate Expression::getUses
  Branch: refs/heads/thaines/deprecate_Expression_getUses
  Home:   https://github.com/dyninst/dyninst
  Commit: eda0eb6137426e1ea1f512228b836e0b217b0db0
      https://github.com/dyninst/dyninst/commit/eda0eb6137426e1ea1f512228b836e0b217b0db0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-26 (Sun, 26 Oct 2025)

  Changed paths:
    M dataflowAPI/src/AbslocInterface.C
    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/Ternary.h
    M instructionAPI/src/Immediate.C
    M instructionAPI/src/Operand.C
    M patchAPI/src/PatchBlock.C

  Log Message:
  -----------
  Deprecate Expression::getUses

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

Expression::getUses isn't marked deprecated because the derived AST
classes still need to call 'getUses' because they can't call
'getRegisterUses' because it needs the definitions of those classes.
RegisterAST::getUses isn't marked deprecated because it gets explicitly
called by MultiRegisterAST::getUses.


  Commit: 21ae07ef6071c5c83727ef95b3985ce18af78d23
      https://github.com/dyninst/dyninst/commit/21ae07ef6071c5c83727ef95b3985ce18af78d23
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-10-26 (Sun, 26 Oct 2025)

  Changed paths:
    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

  Log Message:
  -----------
  Update docs


Compare: https://github.com/dyninst/dyninst/compare/a5af2b461b16...21ae07ef6071

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] eda0eb: Deprecate Expression::getUses, Tim Haines <=