[DynInst_API:] [dyninst/dyninst] d04e89: Add newline in log message in DwarfWalker::parseMo...


Date: Wed, 06 Sep 2023 13:06:04 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] d04e89: Add newline in log message in DwarfWalker::parseMo...
  Branch: refs/heads/thaines/unify_symtab_module_naming
  Home:   https://github.com/dyninst/dyninst
  Commit: d04e892c8f3c9451ff4e77187416a391b5e4ecfa
      https://github.com/dyninst/dyninst/commit/d04e892c8f3c9451ff4e77187416a391b5e4ecfa
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M symtabAPI/src/dwarfWalker.C

  Log Message:
  -----------
  Add newline in log message in DwarfWalker::parseModule (#1506)


  Commit: 463801216b933cdfdc4115b3d170eb4f64b513f1
      https://github.com/dyninst/dyninst/commit/463801216b933cdfdc4115b3d170eb4f64b513f1
  Author: bbiiggppiigg <bbiiggppiigg@xxxxxxxxx>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M common/h/dyn_regs.h
    M instructionAPI/h/Immediate.h
    M instructionAPI/h/Instruction.h
    M instructionAPI/h/Operation_impl.h
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.C
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.h
    M instructionAPI/src/AMDGPU/gfx908/amdgpu_gfx908_decoder_impl.C
    M instructionAPI/src/AMDGPU/gfx908/amdgpu_gfx908_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx908/decodeOperands.C
    M instructionAPI/src/AMDGPU/gfx908/finalizeOperands.C
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.C
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.h
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_decoder_impl.C
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx90a/decodeOperands.C
    M instructionAPI/src/AMDGPU/gfx90a/finalizeOperands.C
    M instructionAPI/src/Immediate.C

  Log Message:
  -----------
  Workarounds to align decoder output with llvm-objdump (#1508)

* Add info registers and fix some duplicate values

* Add named immediate as an operand type

* Add a method for updating mnemonic(required for adding extensions)

* Implement Workarounds for Gfx908 and GFX90A

This commit is intended to fix the following problems

1. Instructions that use SDWA have wrong length
2. Instructions that use SDWA does not have the _SDWA suffixh
3. MUBUF instructions decodes the VADDR field incorrectly
4. MUBUF instructions missing information about offset
5. DS instructions missing information about offset

* redo whitespace and indentation changes


  Commit: 481ee344d36721c44aa456e31937836c90ee235d
      https://github.com/dyninst/dyninst/commit/481ee344d36721c44aa456e31937836c90ee235d
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M symtabAPI/doc/API/Symtab/Function.tex
    M symtabAPI/doc/API/Symtab/FunctionBase.tex
    M symtabAPI/h/Function.h

  Log Message:
  -----------
  Remove FunctionBase::setReturnType(Type*) (#1503)

This is never called from within Dyninst and should never be called by
a user.


  Commit: 7acb76787c647a434301579e4e9baa3a59884eb4
      https://github.com/dyninst/dyninst/commit/7acb76787c647a434301579e4e9baa3a59884eb4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M symtabAPI/src/dwarfWalker.C

  Log Message:
  -----------
  Don't check for existing return type when parsing DWARF subprogram (#1509)

Since 8b400af5, functions are guaranteed to only be parsed once, so the
return type can never be set before the call here. Checking the return
type circularly invokes Symtab::parseTypesNow which deadlocks since
7f1e24d.


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

  Changed paths:
    M dwarf/h/dwarf_cu_info.hpp

  Log Message:
  -----------
  Fix inverted logic  in DwarfDyninst::is_cudie (#1505)

If a DIE is a CU, then it contains a reference to its Dwarf_CU.

Introduced by 71f2e14d3 (PR1499).


  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: 1763b899c090f70444fb90685539de73dda3e7d7
      https://github.com/dyninst/dyninst/commit/1763b899c090f70444fb90685539de73dda3e7d7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M common/h/dyn_regs.h
    M common/src/MappedFile.C
    M common/src/MappedFile.h
    A dwarf/h/dwarf_cu_info.hpp
    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 instructionAPI/h/Immediate.h
    M instructionAPI/h/Instruction.h
    M instructionAPI/h/Operation_impl.h
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.C
    M instructionAPI/src/AMDGPU/gfx908/InstructionDecoder-amdgpu-gfx908.h
    M instructionAPI/src/AMDGPU/gfx908/amdgpu_gfx908_decoder_impl.C
    M instructionAPI/src/AMDGPU/gfx908/amdgpu_gfx908_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx908/decodeOperands.C
    M instructionAPI/src/AMDGPU/gfx908/finalizeOperands.C
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.C
    M instructionAPI/src/AMDGPU/gfx90a/InstructionDecoder-amdgpu-gfx90a.h
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_decoder_impl.C
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx90a/decodeOperands.C
    M instructionAPI/src/AMDGPU/gfx90a/finalizeOperands.C
    M instructionAPI/src/Immediate.C
    M symtabAPI/doc/API/Symtab/Function.tex
    M symtabAPI/doc/API/Symtab/FunctionBase.tex
    M symtabAPI/h/Function.h
    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:
  -----------
  Merge branch 'master' into thaines/unify_symtab_module_naming


Compare: https://github.com/dyninst/dyninst/compare/e915c9fea0f7...1763b899c090
[← Prev in Thread] Current Thread [Next in Thread→]