[DynInst_API:] [dyninst/dyninst] 19a8ab: remove LineInformation::addLineInfo


Date: Mon, 14 Apr 2025 09:48:19 -0700
From: kupsch <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 19a8ab: remove LineInformation::addLineInfo
  Branch: refs/heads/kupsch/lineinfo-improvements
  Home:   https://github.com/dyninst/dyninst
  Commit: 19a8abba5a88cdfd012bbac8e16039eaf6abae5a
      https://github.com/dyninst/dyninst/commit/19a8abba5a88cdfd012bbac8e16039eaf6abae5a
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-11 (Fri, 11 Apr 2025)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/src/LineInformation.C

  Log Message:
  -----------
  remove LineInformation::addLineInfo


  Commit: 28eb93751d5741a7028ed6e05b5006da6cacfee7
      https://github.com/dyninst/dyninst/commit/28eb93751d5741a7028ed6e05b5006da6cacfee7
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-11 (Fri, 11 Apr 2025)

  Changed paths:
    M dyninstAPI/h/BPatch.h
    M dyninstAPI/src/BPatch.C
    M symtabAPI/h/Symtab.h
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/Object-elf.h
    M symtabAPI/src/Object.C
    M symtabAPI/src/Object.h
    M symtabAPI/src/Symtab.C

  Log Message:
  -----------
  remove truncateLineInfoFilenames


  Commit: c5f59a3a479492c78fa5582c348b76181a5a89a4
      https://github.com/dyninst/dyninst/commit/c5f59a3a479492c78fa5582c348b76181a5a89a4
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-11 (Fri, 11 Apr 2025)

  Changed paths:
    M symtabAPI/h/LineInformation.h

  Log Message:
  -----------
  reformat LineInformation.h


  Commit: 4d7004eb5f1990201a208a5fb277ec942b663443
      https://github.com/dyninst/dyninst/commit/4d7004eb5f1990201a208a5fb277ec942b663443
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-14 (Mon, 14 Apr 2025)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/h/Module.h
    M symtabAPI/h/StringTable.h
    M symtabAPI/src/LineInformation.C
    M symtabAPI/src/Module.C
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/Object-elf.h
    M symtabAPI/src/Object.h

  Log Message:
  -----------
  make StringTable a member of LineInformation

- remove LineInformation::setStrings method
- remove StringTablePtr parameters from Object::parseLineInfoForObject


  Commit: 120fe670d4702a442b4c4516efe6360932ff25af
      https://github.com/dyninst/dyninst/commit/120fe670d4702a442b4c4516efe6360932ff25af
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-14 (Mon, 14 Apr 2025)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/src/LineInformation.C
    M symtabAPI/src/Object-elf.h
    M symtabAPI/src/Object.h

  Log Message:
  -----------
  LineInformation: add init, locking & lock guards

- add initialize method that intialize LineInformation from Module or
  Object as appropriate

- add multiple reader, single writer locking that by default initizes
  the LineInformation safely as part of acquiring the lock

- add read and write lock guard classes


  Commit: 8b56f3819e5190f78c8b54583ef694466321f6cd
      https://github.com/dyninst/dyninst/commit/8b56f3819e5190f78c8b54583ef694466321f6cd
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-14 (Mon, 14 Apr 2025)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/h/Module.h
    M symtabAPI/src/Module.C
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/Object-elf.h
    M symtabAPI/src/Object.h

  Log Message:
  -----------
  update Module and Object-elf LineInformation usage

- rewrite parseLineInformation to use new locking/initialization
- make Object-elf and Module contain a LineInformation object
  instead of a pointer
- remove default Modules default constructor, copy constructor
  operator==, and setLineInfo


  Commit: 27d2cc385e845e7f53e7a191b7a12aaf27d754f8
      https://github.com/dyninst/dyninst/commit/27d2cc385e845e7f53e7a191b7a12aaf27d754f8
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-14 (Mon, 14 Apr 2025)

  Changed paths:
    M common/CMakeLists.txt
    A common/h/DynIntervalMap.h

  Log Message:
  -----------
  add DynIntervalMap.h


  Commit: 6aee1d040a0f6770abf41f3b74234de56007369d
      https://github.com/dyninst/dyninst/commit/6aee1d040a0f6770abf41f3b74234de56007369d
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-14 (Mon, 14 Apr 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_statement.h
    M dyninstAPI/src/BPatch_module.C
    M dyninstAPI/src/BPatch_statement.C
    M symtabAPI/h/LineInformation.h
    M symtabAPI/h/Module.h
    M symtabAPI/h/Symtab.h
    M symtabAPI/src/LineInformation.C
    M symtabAPI/src/Module.C
    M symtabAPI/src/Symtab.C

  Log Message:
  -----------
  use DynIntervalMap in LineInformation

- Add a StatementIntervalMap to LineInformation
- BPatch_statement now has a copy of the statement instead of a ptr
- remove methods that return vectors of Statement::Ptr, use vector
  of Statement instead
- Add getSourceLines(addr) that returns the statement vector
- set macro NEW_GETSOURCELINES_INTERFACE to 2


  Commit: 7dd349f4c60b717ac1134b91374ee729be319180
      https://github.com/dyninst/dyninst/commit/7dd349f4c60b717ac1134b91374ee729be319180
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2025-04-14 (Mon, 14 Apr 2025)

  Changed paths:
    M symtabAPI/src/LineInformation.C

  Log Message:
  -----------
  Add locking to more LineInformation methods


Compare: https://github.com/dyninst/dyninst/compare/19a8abba5a88%5E...7dd349f4c60b

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] 19a8ab: remove LineInformation::addLineInfo, kupsch <=