Skip to content

Dyninst build - Refactor elfutils build folders#238

Merged
dgaliffiAMD merged 6 commits intoROCm:amd-stagingfrom
dgaliffiAMD:dyninst-cmake
Jun 13, 2025
Merged

Dyninst build - Refactor elfutils build folders#238
dgaliffiAMD merged 6 commits intoROCm:amd-stagingfrom
dgaliffiAMD:dyninst-cmake

Conversation

@dgaliffiAMD
Copy link
Collaborator

@dgaliffiAMD dgaliffiAMD commented Jun 9, 2025

  • Set elfutils build folder to external/elfutils, to match other external dependencies
  • Update warning message when using the "DYNINST_BUILD_" style options to make it clearer that the new options are still being set.
  • Declare interfaces for the Dyninst build dependencies with the other "External Packages" in Packages.cmake.

@dgaliffiAMD dgaliffiAMD requested review from a team and jrmadsen as code owners June 9, 2025 16:46
Update the deprecation warning for "DYNINST_BUILD_${dep}" to make it clearer that the option is being translated to the new option, "ROCPROFSYS_BUILD_${dep}"
@dgaliffiAMD dgaliffiAMD requested a review from pranswarup June 10, 2025 23:47
@dgaliffiAMD dgaliffiAMD merged commit eaaec2c into ROCm:amd-staging Jun 13, 2025
41 of 46 checks passed
@dgaliffiAMD dgaliffiAMD deleted the dyninst-cmake branch June 13, 2025 21:19
ammallya pushed a commit that referenced this pull request Oct 28, 2025
- Refactor elfutils build folders. Move build artifacts to the "external" subfolders
- Remove reference to unused DYNINST configuration
- Declare Dyninst interfaces with other external declarations
- Updated cmake warning message
- Update the deprecation warning for "DYNINST_BUILD_${dep}" to make it clearer that the option is being translated to the new option, "ROCPROFSYS_BUILD_${dep}"

[ROCm/rocprofiler-systems commit: eaaec2c]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant