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

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

Date:	Wed, 08 Oct 2025 13:02:53 -0700
From:	Ronak Chauhan <noreply@xxxxxxxxxx>
Subject:	[DynInst_API:] [dyninst/dyninst] e4a247: Cleaned up initial commit for AMDGPU Codegen

  Branch: refs/heads/ronak/amdgpu-review
  Home:   https://github.com/dyninst/dyninst
  Commit: e4a247fef954cc6ae8ce4b2260d4c008502bb651
      https://github.com/dyninst/dyninst/commit/e4a247fef954cc6ae8ce4b2260d4c008502bb651
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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-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
    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: 300b150057c0826fb6cb007ba2feb68a607a2460
      https://github.com/dyninst/dyninst/commit/300b150057c0826fb6cb007ba2feb68a607a2460
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

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

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


  Commit: b36476814d32c0166aee5400f67ab6e2bddbf904
      https://github.com/dyninst/dyninst/commit/b36476814d32c0166aee5400f67ab6e2bddbf904
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

  Changed paths:
    M dyninstAPI/CMakeLists.txt

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


  Commit: 88685157bf7f17138e5269023ebba6d725d4f1ef
      https://github.com/dyninst/dyninst/commit/88685157bf7f17138e5269023ebba6d725d4f1ef
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

  Changed paths:
    M dataflowAPI/src/ABI.C

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


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

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

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


  Commit: 3b1555e99e1f900f70adf35342ee1227e80bd27c
      https://github.com/dyninst/dyninst/commit/3b1555e99e1f900f70adf35342ee1227e80bd27c
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 1fd8247de1b24115c53281c16146c97b87537415
      https://github.com/dyninst/dyninst/commit/1fd8247de1b24115c53281c16146c97b87537415
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: bc85e02ca728ee8cf669b95eefa4a881672e66bb
      https://github.com/dyninst/dyninst/commit/bc85e02ca728ee8cf669b95eefa4a881672e66bb
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 431dbab579a6398c4212bd225bf412c5a50d04ca
      https://github.com/dyninst/dyninst/commit/431dbab579a6398c4212bd225bf412c5a50d04ca
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 7d5230c9c9d05103ca103c0c6b8b25a829902f9c
      https://github.com/dyninst/dyninst/commit/7d5230c9c9d05103ca103c0c6b8b25a829902f9c
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 0dab339dfe6fecf989bf6e7197200307573b7748
      https://github.com/dyninst/dyninst/commit/0dab339dfe6fecf989bf6e7197200307573b7748
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

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

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


  Commit: 5e356206cef78c8ed8bc98a133c10bdc3357c0b8
      https://github.com/dyninst/dyninst/commit/5e356206cef78c8ed8bc98a133c10bdc3357c0b8
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 6b621205c044dafdb7c33374866e5574876cb2fb
      https://github.com/dyninst/dyninst/commit/6b621205c044dafdb7c33374866e5574876cb2fb
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: ece23e8828c3df0819a8990013fec4589c18364f
      https://github.com/dyninst/dyninst/commit/ece23e8828c3df0819a8990013fec4589c18364f
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 5f7646d6c868678f19e9f7ecfc8a5563eee9bba5
      https://github.com/dyninst/dyninst/commit/5f7646d6c868678f19e9f7ecfc8a5563eee9bba5
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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: 92cace185f84e823d81f0d679609cba62a0feca3
      https://github.com/dyninst/dyninst/commit/92cace185f84e823d81f0d679609cba62a0feca3
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

  Changed paths:
    M dyninstAPI/src/BPatch_binaryEdit.C

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


  Commit: 07e824eea2e8afbfb79f9ee4f0a7419a679d428f
      https://github.com/dyninst/dyninst/commit/07e824eea2e8afbfb79f9ee4f0a7419a679d428f
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

  Changed paths:
    M dyninstAPI/src/registerSpace.C

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


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

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

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


  Commit: 39405776cf31ca8796533fdc8c68d9c0e6cbe555
      https://github.com/dyninst/dyninst/commit/39405776cf31ca8796533fdc8c68d9c0e6cbe555
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_snippet.h

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


  Commit: c7105af0ad59a7a04461f208ca32d96138942aaa
      https://github.com/dyninst/dyninst/commit/c7105af0ad59a7a04461f208ca32d96138942aaa
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-10-08 (Wed, 08 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


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

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

  Log Message:
  -----------
  Remove redundant amdgpu-gfx908-details.cpp after rebasing


Compare: https://github.com/dyninst/dyninst/compare/dcf453f8b975...dd8f6898bd8c

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

Previous by Date:	[DynInst_API:] [dyninst/dyninst] 2b05a8: [AMDGPU] Add emitter implementation for gfx908 (#2..., Ronak Chauhan
Next by Date:	[DynInst_API:] [dyninst/dyninst] c8ef60: Add support for prologue/epilogue snippet types an..., Ronak Chauhan
Previous by Thread:	[DynInst_API:] [dyninst/dyninst] e4888f: Move hasFrameDebugInfo, getRegValueAtFrame into Ob..., Tim Haines
Next by Thread:	[DynInst_API:] [dyninst/dyninst] e5029c: Import unit tests, Tim Haines
Indexes:	[Date] [Thread]

Mailing List Archives

Authenticated access

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