[DynInst_API:] [dyninst/dyninst] 845f93: remove singleton_object_pool (#1789)


Date: Mon, 07 Oct 2024 14:15:43 -0700
From: kupsch <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 845f93: remove singleton_object_pool (#1789)
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 845f93bfe43aba2060a803fcd11c84101b1657ca
      https://github.com/dyninst/dyninst/commit/845f93bfe43aba2060a803fcd11c84101b1657ca
  Author: kupsch <kupsch@xxxxxxxxxxx>
  Date:   2024-10-07 (Mon, 07 Oct 2024)

  Changed paths:
    M common/CMakeLists.txt
    M common/src/AST.C
    R common/src/pool_allocators.h
    R common/src/singleton_object_pool.h
    M dataflowAPI/src/Absloc.C
    M dataflowAPI/src/AbslocInterface.C
    M dataflowAPI/src/RoseInsnFactory.C
    M instructionAPI/src/Immediate.C
    M instructionAPI/src/Instruction.C
    M instructionAPI/src/InstructionDecoder-power.C
    M instructionAPI/src/InstructionDecoder-x86.C
    M instructionAPI/src/InstructionDecoderImpl.C
    M instructionAPI/src/MultiRegister.C
    M instructionAPI/src/Operation.C
    M instructionAPI/src/Register.C
    M instructionAPI/src/Ternary.C

  Log Message:
  -----------
  remove singleton_object_pool (#1789)

- All uses of singleton_object_pool use the standard allocator,
  so this serves no purpose

- replace uses of the function ::make_shared with boost::make_shared
  as ::make_shared can leak if exceptions are thrown, and it requires
  two memory allocation for the shared_ptr's metadata and the object
  itself where boost::make_shared combines them into one

- remove pool_allocators.h as it is unused



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] 845f93: remove singleton_object_pool (#1789), kupsch <=