[DynInst_API:] [dyninst/dyninst] fb799f: Cleaned up initial commit for AMDGPU Codegen

Mailing List Archives Authenticated access	UW Madison Computer Sciences Department Computer Systems Lab

Date:	Thu, 25 Sep 2025 11:28:59 -0700
From:	bbiiggppiigg <noreply@xxxxxxxxxx>
Subject:	[DynInst_API:] [dyninst/dyninst] fb799f: Cleaned up initial commit for AMDGPU Codegen

  Branch: refs/heads/bbiiggppiigg/amdgpu-review-getpoint
  Home:   https://github.com/dyninst/dyninst
  Commit: fb799fbdf982141ab4e4e3e7b8a4adf10505ff99
      https://github.com/dyninst/dyninst/commit/fb799fbdf982141ab4e4e3e7b8a4adf10505ff99
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M cmake/DyninstCapArchDef.cmake
    M common/CMakeLists.txt
    A common/src/arch-amdgpu.C
    A common/src/arch-amdgpu.h
    M dataflowAPI/src/ABI.C
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/h/BPatch_addressSpace.h
    M dyninstAPI/h/BPatch_snippet.h
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_binaryEdit.C
    M dyninstAPI/src/BPatch_memoryAccessAdapter.C
    M dyninstAPI/src/BPatch_snippet.C
    A dyninstAPI/src/RegisterConversion-amdgpu.C
    A dyninstAPI/src/Relocation/Widgets/CFWidget-amdgpu.C
    M dyninstAPI/src/Relocation/Widgets/CFWidget.h
    A dyninstAPI/src/Relocation/Widgets/PCWidget-amdgpu.C
    A dyninstAPI/src/amdgpu-epilogue.cpp
    A dyninstAPI/src/amdgpu-epilogue.h
    A dyninstAPI/src/amdgpu-gfx908-details.cpp
    A dyninstAPI/src/amdgpu-gfx908-details.h
    A dyninstAPI/src/amdgpu-prologue.cpp
    A dyninstAPI/src/amdgpu-prologue.h
    A dyninstAPI/src/ast-amdgpu.C
    M dyninstAPI/src/ast.C
    M dyninstAPI/src/ast.h
    M dyninstAPI/src/binaryEdit.C
    A dyninstAPI/src/codegen-amdgpu.C
    A dyninstAPI/src/codegen-amdgpu.h
    M dyninstAPI/src/codegen.C
    M dyninstAPI/src/codegen.h
    A dyninstAPI/src/emit-amdgpu.C
    A dyninstAPI/src/emit-amdgpu.h
    M dyninstAPI/src/function.h
    A dyninstAPI/src/inst-amdgpu.C
    A dyninstAPI/src/inst-amdgpu.h
    M dyninstAPI/src/mapped_object.C
    A dyninstAPI/src/parse-amdgpu.C
    M dyninstAPI/src/parse-cfg.h
    M dyninstAPI/src/registerSpace.C
    M dyninstAPI/src/registerSpace.h
    M symtabAPI/CMakeLists.txt
    A symtabAPI/src/emitElfStatic-amdgpu.C
    A symtabAPI/src/relocationEntry-elf-amdgpu.C

  Log Message:
  -----------
  Cleaned up initial commit for AMDGPU Codegen


  Commit: 23889131e74be202866bd2ff684bbacb02782da9
      https://github.com/dyninst/dyninst/commit/23889131e74be202866bd2ff684bbacb02782da9
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M cmake/DyninstCapArchDef.cmake
    M dyninstAPI/CMakeLists.txt

  Log Message:
  -----------
  run cmake-format


  Commit: d063b3f0d02d9b9773473f549fd9c0a5b84b5d46
      https://github.com/dyninst/dyninst/commit/d063b3f0d02d9b9773473f549fd9c0a5b84b5d46
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt

  Log Message:
  -----------
  remove redundant ast-amdgpu.C


  Commit: b09e7516c812abeb47ff47dbc90ce34f1c93caab
      https://github.com/dyninst/dyninst/commit/b09e7516c812abeb47ff47dbc90ce34f1c93caab
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dataflowAPI/src/ABI.C

  Log Message:
  -----------
  fix amdgpu abi initilization


  Commit: 08f0d2654e96ba05feee2c8035d7896d31ac5111
      https://github.com/dyninst/dyninst/commit/08f0d2654e96ba05feee2c8035d7896d31ac5111
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/inst-amdgpu.C
    M dyninstAPI/src/inst-amdgpu.h

  Log Message:
  -----------
  [AMDGPU] Cleanup some dead code


  Commit: db403ca81d0fe23c19b1a748e6d495d76da9d74d
      https://github.com/dyninst/dyninst/commit/db403ca81d0fe23c19b1a748e6d495d76da9d74d
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/function.h
    M dyninstAPI/src/parse-cfg.h
    M dyninstAPI/src/registerSpace.C

  Log Message:
  -----------
  Correct some conditional compilations


  Commit: 08b30e9e0a15653631aab0e6cc6cbc5f25b622ed
      https://github.com/dyninst/dyninst/commit/08b30e9e0a15653631aab0e6cc6cbc5f25b622ed
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/linux-aarch64.C
    M dyninstAPI/src/linux-power.C

  Log Message:
  -----------
  Temporary fix for host-codegen separation on Power and AArch64


  Commit: 893c0b6c2d41d536bd7ff22167840ff8d8f6a1d7
      https://github.com/dyninst/dyninst/commit/893c0b6c2d41d536bd7ff22167840ff8d8f6a1d7
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

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

  Log Message:
  -----------
  [AMDGPU] Use existing constructor for variables


  Commit: f20ea781e1887822daf9f3591320c6ebcee482c4
      https://github.com/dyninst/dyninst/commit/f20ea781e1887822daf9f3591320c6ebcee482c4
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

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

  Log Message:
  -----------
  Cleanup accumulated minor diffs in BPatch_snippet


  Commit: 1b435282631a35755ae0cdf4ba9fb68fe3893817
      https://github.com/dyninst/dyninst/commit/1b435282631a35755ae0cdf4ba9fb68fe3893817
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt
    M dyninstAPI/src/BPatch_snippet.C
    M dyninstAPI/src/ast-amdgpu.C
    M dyninstAPI/src/ast.C
    R dyninstAPI/src/inst-amdgpu.h
    M dyninstAPI/src/registerSpace.C

  Log Message:
  -----------
  Delete inst-amdgpu.h


  Commit: 59bc8dc8df8a89ae06ad5ecc69cb9b064022d99b
      https://github.com/dyninst/dyninst/commit/59bc8dc8df8a89ae06ad5ecc69cb9b064022d99b
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_addressSpace.h
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_binaryEdit.C
    M dyninstAPI/src/amdgpu-prologue.cpp
    M dyninstAPI/src/amdgpu-prologue.h

  Log Message:
  -----------
  [AMDGPU] Simplify and cleanup prologue implementation


  Commit: a8c3c3bad02248889780ff82ff8554dc441b6530
      https://github.com/dyninst/dyninst/commit/a8c3c3bad02248889780ff82ff8554dc441b6530
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/amdgpu-gfx908-details.cpp
    M dyninstAPI/src/amdgpu-gfx908-details.h

  Log Message:
  -----------
  [AMDGPU] Fix s_dcache_wb generation


  Commit: 9533c3fa119a3e3488365230ae82dc79859d26cd
      https://github.com/dyninst/dyninst/commit/9533c3fa119a3e3488365230ae82dc79859d26cd
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_addressSpace.h
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_binaryEdit.C
    M dyninstAPI/src/amdgpu-epilogue.cpp
    M dyninstAPI/src/amdgpu-epilogue.h

  Log Message:
  -----------
  [AMDGPU] Simplify and cleanup epilogue implementation


  Commit: fa34c8047701410140e96d3efdfeacb5783d01e3
      https://github.com/dyninst/dyninst/commit/fa34c8047701410140e96d3efdfeacb5783d01e3
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/amdgpu-prologue.h
    M dyninstAPI/src/codegen-amdgpu.C
    M dyninstAPI/src/inst-amdgpu.C

  Log Message:
  -----------
  Change comments


  Commit: 54e29647820fce4931b436b99e62df23ba16014e
      https://github.com/dyninst/dyninst/commit/54e29647820fce4931b436b99e62df23ba16014e
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_snippet.h
    M dyninstAPI/src/BPatch_addressSpace.C
    M dyninstAPI/src/BPatch_snippet.C

  Log Message:
  -----------
  [AMDGPU] Inline the special variant of makeVariableExpr


  Commit: 6ac42c57c1a94510a77bc07cb405814ee0c2cfe6
      https://github.com/dyninst/dyninst/commit/6ac42c57c1a94510a77bc07cb405814ee0c2cfe6
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M common/src/arch-amdgpu.C
    M common/src/arch-amdgpu.h
    M dyninstAPI/src/codegen-amdgpu.C
    M dyninstAPI/src/codegen.C
    M dyninstAPI/src/inst-amdgpu.C

  Log Message:
  -----------
  [AMDGPU] Cleanup comments


  Commit: 667c0f06ffedefce55e31dcbb6e0f82eea695837
      https://github.com/dyninst/dyninst/commit/667c0f06ffedefce55e31dcbb6e0f82eea695837
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/BPatch_binaryEdit.C

  Log Message:
  -----------
  [AMDGPU] Don't use insertSnippet for prologue and epilogue


  Commit: d46fbf96f592909492de281a4bee6c26f1b5e2f9
      https://github.com/dyninst/dyninst/commit/d46fbf96f592909492de281a4bee6c26f1b5e2f9
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

  Changed paths:
    M dyninstAPI/src/registerSpace.C

  Log Message:
  -----------
  Remove unnecessary #ifdefs for x86


  Commit: afb4167ccda0f53ab00838ef5b2e312fda1f953b
      https://github.com/dyninst/dyninst/commit/afb4167ccda0f53ab00838ef5b2e312fda1f953b
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-09-25 (Thu, 25 Sep 2025)

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

  Log Message:
  -----------
  demonstrate getpoint is fine at it is


Compare: https://github.com/dyninst/dyninst/compare/e9ba49536acb...afb4167ccda0

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] fb799f: Cleaned up initial commit for AMDGPU Codegen, bbiiggppiigg <=

Previous by Date:	[DynInst_API:] [dyninst/dyninst] 6c3bf5: Fix default implementation in emitElfStatic-stub, bbiiggppiigg
Next by Date:	[DynInst_API:] [dyninst/dyninst] 09b893: stats_codegen has been removed, bbiiggppiigg
Previous by Thread:	[DynInst_API:] [dyninst/dyninst] fb4548: Split codegen into multiple of 16 bits, wxrdnx
Next by Thread:	[DynInst_API:] [dyninst/dyninst] fb7b30: Add riscv64.h in dwarf/CMakeLists.txt, wxrdnx
Indexes:	[Date] [Thread]

Mailing List Archives

Authenticated access

[DynInst_API:] [dyninst/dyninst] fb799f: Cleaned up initial commit for AMDGPU Codegen