[DynInst_API:] [dyninst/dyninst] 6eec48: Use full filenames when handling Modules (#1501)


Date: Wed, 06 Sep 2023 13:17:30 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 6eec48: Use full filenames when handling Modules (#1501)
  Branch: refs/heads/thaines/remove_symtab_function_lock
  Home:   https://github.com/dyninst/dyninst
  Commit: 6eec4844b82ed2176c455359efb29c5ca04a3e87
      https://github.com/dyninst/dyninst/commit/6eec4844b82ed2176c455359efb29c5ca04a3e87
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M common/src/MappedFile.C
    M common/src/MappedFile.h
    M dyninstAPI/src/BPatch_module.C
    M dyninstAPI/src/image.C
    M dyninstAPI/src/image.h
    M dyninstAPI/src/mapped_module.C
    M dyninstAPI/src/mapped_module.h
    M dyninstAPI/src/mapped_object.C
    M dyninstAPI/src/unix.C
    M symtabAPI/h/Module.h
    M symtabAPI/h/Symtab.h
    M symtabAPI/src/Aggregate.C
    M symtabAPI/src/Archive.C
    M symtabAPI/src/Module.C
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/Symbol.C
    M symtabAPI/src/Symtab-lookup.C
    M symtabAPI/src/Symtab.C
    M symtabAPI/src/dwarfWalker.C

  Log Message:
  -----------
  Use full filenames when handling Modules (#1501)

A module's name is either a unique string or the pathname of the file
used to create the corresponding DWARF compilation unit. In the latter
case, the 'name' is only the basename of the path which is never
guaranteed to be unique (e.g., /path1/foo.c /path2/foo.c would have the
same 'name').

* Remove useless file creations

These don't correspond to any Module.


  Commit: 212ccd3b4e12f52cb6a743f1fbeff9fccfe7d0bc
      https://github.com/dyninst/dyninst/commit/212ccd3b4e12f52cb6a743f1fbeff9fccfe7d0bc
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-06 (Wed, 06 Sep 2023)

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

  Log Message:
  -----------
  Remove hasSpecification param from DwarfWalker::getReturnType (#1507)

Since 5ed068167 in 2017, only literal 'false' has ever been used.


  Commit: d129555b064092d18b69b4b22240234c20432168
      https://github.com/dyninst/dyninst/commit/d129555b064092d18b69b4b22240234c20432168
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-06 (Wed, 06 Sep 2023)

  Changed paths:
    M common/src/MappedFile.C
    M common/src/MappedFile.h
    M dyninstAPI/src/BPatch_module.C
    M dyninstAPI/src/image.C
    M dyninstAPI/src/image.h
    M dyninstAPI/src/mapped_module.C
    M dyninstAPI/src/mapped_module.h
    M dyninstAPI/src/mapped_object.C
    M dyninstAPI/src/unix.C
    M symtabAPI/h/Module.h
    M symtabAPI/h/Symtab.h
    M symtabAPI/src/Aggregate.C
    M symtabAPI/src/Archive.C
    M symtabAPI/src/Module.C
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/Symbol.C
    M symtabAPI/src/Symtab-lookup.C
    M symtabAPI/src/Symtab.C
    M symtabAPI/src/dwarfWalker.C
    M symtabAPI/src/dwarfWalker.h

  Log Message:
  -----------
  Merge branch 'master' into thaines/remove_symtab_function_lock


Compare: https://github.com/dyninst/dyninst/compare/8af1f578be26...d129555b0640
[← Prev in Thread] Current Thread [Next in Thread→]