[DynInst_API:] [dyninst/dyninst] 35d8bb: Remove image::main_call_addr_

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/thaines/cleanup_findMain
  Home:   https://github.com/dyninst/dyninst
  Commit: 35d8bb6ae0f5795f8cd82e352a5daa45810dbf22
      https://github.com/dyninst/dyninst/commit/35d8bb6ae0f5795f8cd82e352a5daa45810dbf22
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.h

  Log Message:
  -----------
  Remove image::main_call_addr_


  Commit: 87ee4ba522a618e4b165ed431da2ca07a9d63350
      https://github.com/dyninst/dyninst/commit/87ee4ba522a618e4b165ed431da2ca07a9d63350
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.C
    M dyninstAPI/src/image.h

  Log Message:
  -----------
  Add image::getAddressOfMain


  Commit: 70eb99cf26c6c7a37423a5b32c77f2f514af0afb
      https://github.com/dyninst/dyninst/commit/70eb99cf26c6c7a37423a5b32c77f2f514af0afb
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Ensure file is an executable


  Commit: 1c0ea07db3ebc7a0a63a74ea566227ad034f67d9
      https://github.com/dyninst/dyninst/commit/1c0ea07db3ebc7a0a63a74ea566227ad034f67d9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Require the presence of at least one code region

For ppc, this is effectively the same as requiring the .text section.


  Commit: a81bfb1caf04b8670ba5b4e1560e52899ec7154e
      https://github.com/dyninst/dyninst/commit/a81bfb1caf04b8670ba5b4e1560e52899ec7154e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Uniformly check for 'main' symbol name


  Commit: 6514c9ef239e27375fd2d75584340d8b42a94547
      https://github.com/dyninst/dyninst/commit/6514c9ef239e27375fd2d75584340d8b42a94547
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Add extra logging


  Commit: ee439650b05873a2f1c40a70921487cbdebe45a0
      https://github.com/dyninst/dyninst/commit/ee439650b05873a2f1c40a70921487cbdebe45a0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Unify entry region construction

This is a slight change for PPC which previously parsed the entire
binary. There's no reason to do that since we know the only region we
need is near the entry point. Additionally, PPC allowed for an entry
region with no calls. That is inconsistent with the libc startup
procedure, so it's removed.


  Commit: d875d6515e1b3da5140f37526317cf1990a7682d
      https://github.com/dyninst/dyninst/commit/d875d6515e1b3da5140f37526317cf1990a7682d
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    A dyninstAPI/src/find_main.h
    A dyninstAPI/src/find_main_ppc.C
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Extract, refactor ppc lookup

    This also removes deref_odp since it was part of the ELF V1 ABI for
    big-endian PPC that we no longer support. ppc64le uses ELF V2 that
    doesn't have function descriptors (and hence no .odp section).

    64-Bit ELF V2 ABI Specification
    Power Architecture Workgroup Specification
    Revision 1.5
    December 1, 2020
    Section 3.1 ELF Header

    E_flags defining the ABI level:

    0  For ELF object files of an unspecified nature.

    1  For the Power ELF V1 ABI using function descriptors. This ABI is
     currently only used for big-endian PowerPC implementations.

    2  For the OpenPOWER ELF V2 ABI using the facilities described here
and
     including function pointers to directly reference functions.


  Commit: ae3b646a37a7f55fbca3088319d31118e3e59960
      https://github.com/dyninst/dyninst/commit/ae3b646a37a7f55fbca3088319d31118e3e59960
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/find_main.h
    A dyninstAPI/src/find_main_x86.C
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Extract, refactor x86 lookup

The heuristic code for FreeBSD hasn't compiled since at least 2009, so
I dropped it.


  Commit: fcc05c172cb874d5e361ec17a63a4bbb77bf4d8e
      https://github.com/dyninst/dyninst/commit/fcc05c172cb874d5e361ec17a63a4bbb77bf4d8e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    A dyninstAPI/src/find_main.C
    M dyninstAPI/src/find_main.h
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Extract find_main into separate module


  Commit: 4b854886da2a7bc4179610ae463b2cbfc6cc07e7
      https://github.com/dyninst/dyninst/commit/4b854886da2a7bc4179610ae463b2cbfc6cc07e7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/src/find_main.C
    M dyninstAPI/src/find_main.h
    M dyninstAPI/src/image.C

  Log Message:
  -----------
  Extract missing symbol handling


  Commit: 1f5086db790430a7ba5cc5ea9526104e541af422
      https://github.com/dyninst/dyninst/commit/1f5086db790430a7ba5cc5ea9526104e541af422
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/find_main.C
    M dyninstAPI/src/find_main.h
    M dyninstAPI/src/find_main_x86.C
    A dyninstAPI/src/find_main_x86_64.C

  Log Message:
  -----------
  Split x86 and x86_64


  Commit: 6262c4d5c753ff668194ba56cd5bf6b849c23a75
      https://github.com/dyninst/dyninst/commit/6262c4d5c753ff668194ba56cd5bf6b849c23a75
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    M tests/regression/CMakeLists.txt
    A tests/regression/dyninstAPI/CMakeLists.txt
    A tests/regression/dyninstAPI/find_main/CMakeLists.txt
    A tests/regression/dyninstAPI/find_main/find_main.cpp
    A tests/regression/dyninstAPI/find_main/find_main_test.c

  Log Message:
  -----------
  Add tests


  Commit: 7092cfd3c403f9022bd979c5723c326dabc9cc95
      https://github.com/dyninst/dyninst/commit/7092cfd3c403f9022bd979c5723c326dabc9cc95
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-07-20 (Sun, 20 Jul 2025)

  Changed paths:
    A dyninstAPI/src/find_main_aarch64.C

  Log Message:
  -----------
  First impl for aarch64


Compare: https://github.com/dyninst/dyninst/compare/1ea0cb79818c...7092cfd3c403

To unsubscribe from these emails, change your notification settings at https://github.com/dyninst/dyninst/settings/notifications