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

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

Date:	Fri, 03 Oct 2025 09:47:26 -0700
From:	Ronak Chauhan <noreply@xxxxxxxxxx>
Subject:	[DynInst_API:] [dyninst/dyninst] 69427e: Cleaned up initial commit for AMDGPU Codegen

  Branch: refs/heads/ronak/amdgpu-review
  Home:   https://github.com/dyninst/dyninst
  Commit: 69427e5871199373b08e61caf3cb75f69d3ab678
      https://github.com/dyninst/dyninst/commit/69427e5871199373b08e61caf3cb75f69d3ab678
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: f6b04e0ad54b6024977b7fc743bd6940ec52dd25
      https://github.com/dyninst/dyninst/commit/f6b04e0ad54b6024977b7fc743bd6940ec52dd25
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

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

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


  Commit: 4c771fcae087d44f740ca66c822d737b5d9632db
      https://github.com/dyninst/dyninst/commit/4c771fcae087d44f740ca66c822d737b5d9632db
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt

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


  Commit: 4aec41776d9c539340aac296b88c8d91ee1dac85
      https://github.com/dyninst/dyninst/commit/4aec41776d9c539340aac296b88c8d91ee1dac85
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M dataflowAPI/src/ABI.C

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


  Commit: c449e45cd5e5e1db623d0d47aa355d490c702db6
      https://github.com/dyninst/dyninst/commit/c449e45cd5e5e1db623d0d47aa355d490c702db6
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

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

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


  Commit: b027889b70b4805f83f8efcd383408837e3e568c
      https://github.com/dyninst/dyninst/commit/b027889b70b4805f83f8efcd383408837e3e568c
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: 52fed5a7284a1789f86ccc5ed1fb1497b84a7132
      https://github.com/dyninst/dyninst/commit/52fed5a7284a1789f86ccc5ed1fb1497b84a7132
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

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

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


  Commit: 86bd0482b9bde9652f35ba291b4e03338e1b5bb4
      https://github.com/dyninst/dyninst/commit/86bd0482b9bde9652f35ba291b4e03338e1b5bb4
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: e1507579a9e57507c26b8b4726e98e065c99b4c5
      https://github.com/dyninst/dyninst/commit/e1507579a9e57507c26b8b4726e98e065c99b4c5
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: 24bb8f118b7514188d6b88639ddb422b230dae60
      https://github.com/dyninst/dyninst/commit/24bb8f118b7514188d6b88639ddb422b230dae60
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: a8c1baf91654d7408b2f0e0bb1adf0aed0b58d25
      https://github.com/dyninst/dyninst/commit/a8c1baf91654d7408b2f0e0bb1adf0aed0b58d25
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: a556c59d15d159c24d436afa3f3f98f94a6b3b52
      https://github.com/dyninst/dyninst/commit/a556c59d15d159c24d436afa3f3f98f94a6b3b52
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: 07826542bfe5046684732ececadd00caad91516d
      https://github.com/dyninst/dyninst/commit/07826542bfe5046684732ececadd00caad91516d
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: 5781b8c4a396ab4d0efb4540d3ff3aebead903ca
      https://github.com/dyninst/dyninst/commit/5781b8c4a396ab4d0efb4540d3ff3aebead903ca
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: ab340dda01f4411673447f59ef0196133bdce5e4
      https://github.com/dyninst/dyninst/commit/ab340dda01f4411673447f59ef0196133bdce5e4
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 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: 379ca057473843c8764036a88b89c80dfd709665
      https://github.com/dyninst/dyninst/commit/379ca057473843c8764036a88b89c80dfd709665
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M dyninstAPI/src/BPatch_binaryEdit.C

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


  Commit: c3d0e89176475830cfeaf8e822ebc14720b54371
      https://github.com/dyninst/dyninst/commit/c3d0e89176475830cfeaf8e822ebc14720b54371
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M dyninstAPI/src/registerSpace.C

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


  Commit: 9651152e102ed8088d45c049d2b96a101d1f253e
      https://github.com/dyninst/dyninst/commit/9651152e102ed8088d45c049d2b96a101d1f253e
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M dyninstAPI/src/ast-amdgpu.C

  Log Message:
  -----------
  [AMDGPU][AST] Remove unused timer statistics


  Commit: 49af4a0b1bf1b1c4a1cefc30c6699cfbdecbeace
      https://github.com/dyninst/dyninst/commit/49af4a0b1bf1b1c4a1cefc30c6699cfbdecbeace
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_snippet.h

  Log Message:
  -----------
  Remove redundant space


  Commit: dcf453f8b97569534d9e32b9771430ff615ffae0
      https://github.com/dyninst/dyninst/commit/dcf453f8b97569534d9e32b9771430ff615ffae0
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M symtabAPI/CMakeLists.txt
    R symtabAPI/src/emitElfStatic-amdgpu.C
    R symtabAPI/src/relocationEntry-elf-amdgpu.C

  Log Message:
  -----------
  [AMDGPU] Remove SymtabAPI changes


Compare: https://github.com/dyninst/dyninst/compare/078b954c373f...dcf453f8b975

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] 69427e: Cleaned up initial commit for AMDGPU Codegen, Ronak Chauhan <=

Previous by Date:	[DynInst_API:] [dyninst/dyninst] f8ca42: [AMDGPU] Use append_memory_as to write instruction..., Ronak Chauhan
Next by Date:	[DynInst_API:] [dyninst/dyninst] e0ebdd: [AMDGPU] Add emitter implementation for gfx908, Ronak Chauhan
Previous by Thread:	[DynInst_API:] [dyninst/dyninst] 6920dd: Remove parse_func::changeModule, Tim Haines
Next by Thread:	[DynInst_API:] [dyninst/dyninst] 696e83: Don't include platform-specific Object headers in ..., Tim Haines
Indexes:	[Date] [Thread]

Mailing List Archives

Authenticated access

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