[DynInst_API:] [dyninst/dyninst] 705973: [BPatch] Cleanup BPatch

Mailing List Archives Authenticated access	UW Madison Computer Sciences Department Computer Systems Lab
Date:	Fri, 19 Sep 2025 02:06:21 -0700
From:	Ronak Chauhan <noreply@xxxxxxxxxx>
Subject:	[DynInst_API:] [dyninst/dyninst] 705973: [BPatch] Cleanup BPatch_point
  Branch: refs/heads/ronak/amdgpu-review
  Home:   https://github.com/dyninst/dyninst
  Commit: 7059739dfdc06d9c64913b09d63ea655b293ce65
      https://github.com/dyninst/dyninst/commit/7059739dfdc06d9c64913b09d63ea655b293ce65
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 Sep 2025)

  Changed paths:
    M dyninstAPI/h/BPatch_point.h
    M dyninstAPI/src/hybridCallbacks.C
    M dyninstAPI/src/hybridInstrumentation.C
    M dyninstAPI/src/hybridOverwrites.C

  Log Message:
  -----------
  [BPatch] Cleanup BPatch_point

Remove llPoint interface and make getPoint interfaces public


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

  Changed paths:
    M cmake/DyninstCapArchDef.cmake
    M cmake/DyninstPlatform.cmake
    M common/CMakeLists.txt
    A common/src/arch-amdgpu.C
    A common/src/arch-amdgpu.h
    M dataflowAPI/src/ABI.C
    M dataflowAPI/src/RegisterMap.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: 95b3d159046862b9b3d0988e63f94937fea1bac9
      https://github.com/dyninst/dyninst/commit/95b3d159046862b9b3d0988e63f94937fea1bac9
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 Sep 2025)

  Changed paths:
    M dataflowAPI/src/RegisterMap.C

  Log Message:
  -----------
  fix register map for gfx940


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

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

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


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

  Changed paths:
    M dyninstAPI/CMakeLists.txt

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


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

  Changed paths:
    M dataflowAPI/src/ABI.C

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


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

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

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


  Commit: 7f80b8f2c5d83a3c0e8931fa6fd758399257efb2
      https://github.com/dyninst/dyninst/commit/7f80b8f2c5d83a3c0e8931fa6fd758399257efb2
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 8b18531d4fd0969ff2a1bfa45a5f5b978beee3c8
      https://github.com/dyninst/dyninst/commit/8b18531d4fd0969ff2a1bfa45a5f5b978beee3c8
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 3f08716d8e924f7905c0ac17005cee8bc4ebfebe
      https://github.com/dyninst/dyninst/commit/3f08716d8e924f7905c0ac17005cee8bc4ebfebe
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 1f417f7518bcc8b016a7c62702d880866a945a17
      https://github.com/dyninst/dyninst/commit/1f417f7518bcc8b016a7c62702d880866a945a17
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 24f6221bc4f85ad1ffbeb05bc602f2d42889ecf5
      https://github.com/dyninst/dyninst/commit/24f6221bc4f85ad1ffbeb05bc602f2d42889ecf5
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: a352cd4d94334d569bfc9eb15a99aeca27932c7a
      https://github.com/dyninst/dyninst/commit/a352cd4d94334d569bfc9eb15a99aeca27932c7a
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: ec65026d816c70d8b5341925cbf6ccaf651bcb4f
      https://github.com/dyninst/dyninst/commit/ec65026d816c70d8b5341925cbf6ccaf651bcb4f
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 82ee6dcfc1ef068fd6f8d7a736d9ceadd6e21918
      https://github.com/dyninst/dyninst/commit/82ee6dcfc1ef068fd6f8d7a736d9ceadd6e21918
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 7100131ae660d03e5b82731fc02d00826751b566
      https://github.com/dyninst/dyninst/commit/7100131ae660d03e5b82731fc02d00826751b566
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: ccadefde662ba747b27688510023675e13208214
      https://github.com/dyninst/dyninst/commit/ccadefde662ba747b27688510023675e13208214
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: 836bcb01b87f342ac4fcc1e59ae0c991053aadc6
      https://github.com/dyninst/dyninst/commit/836bcb01b87f342ac4fcc1e59ae0c991053aadc6
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-18 (Thu, 18 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: d490e5e9a798561d2db1f2fe278f2645cc07c313
      https://github.com/dyninst/dyninst/commit/d490e5e9a798561d2db1f2fe278f2645cc07c313
  Author: Ronak Chauhan <ronak@xxxxxxxxxxx>
  Date:   2025-09-19 (Fri, 19 Sep 2025)

  Changed paths:
    M dyninstAPI/src/BPatch_binaryEdit.C

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


Compare: https://github.com/dyninst/dyninst/compare/5505a7955d13...d490e5e9a798

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] 705973: [BPatch] Cleanup BPatch_point, Ronak Chauhan <=
Previous by Date:	[DynInst_API:] [dyninst/dyninst] 47ed0b: Add clang-20 to debug frame size override, Tim Haines
Next by Date:	[DynInst_API:] [dyninst/dyninst] 6b2321: [BPatch] Cleanup BPatch_point, Ronak Chauhan
Previous by Thread:	[DynInst_API:] [dyninst/dyninst] 6fc6d5: Add Experimental SAIL parser, wxrdnx
Next by Thread:	[DynInst_API:] [dyninst/dyninst] 717bae: Split tests into groups, Tim Haines
Indexes:	[Date] [Thread]
Mailing List Archives

Authenticated access

[DynInst_API:] [dyninst/dyninst] 705973: [BPatch] Cleanup BPatch_point
