Skip to content

counter_analysis_toolkit: add CMake support#449

Closed
dbarry9 wants to merge 1 commit intoicl-utk-edu:masterfrom
dbarry9:2025.07.14_cmake-for-cat
Closed

counter_analysis_toolkit: add CMake support#449
dbarry9 wants to merge 1 commit intoicl-utk-edu:masterfrom
dbarry9:2025.07.14_cmake-for-cat

Conversation

@dbarry9
Copy link
Contributor

@dbarry9 dbarry9 commented Jul 16, 2025

Pull Request Description

To accommodate different instruction sets and precisions available in modern architectures, these changes add CMake as a more flexible build system for the Counter Analysis Toolkit.

These changes have been tested on the Intel Sapphire Rapids architecture.

Author Checklist

  • Description
    Why this PR exists. Reference all relevant information, including background, issues, test failures, etc
  • Commits
    Commits are self contained and only do one thing
    Commits have a header of the form: module: short description
    Commits have a body (whenever relevant) containing a detailed description of the addressed problem and its solution
  • Tests
    The PR needs to pass all the tests

@dbarry9 dbarry9 force-pushed the 2025.07.14_cmake-for-cat branch 2 times, most recently from 386843c to d14ab64 Compare July 18, 2025 18:04
@dbarry9
Copy link
Contributor Author

dbarry9 commented Jul 22, 2025

@Treece-Burgess I am writing this message just to document that we will need to update the CI tests to account for the new directory structure introduced in this PR.

@dbarry9 dbarry9 force-pushed the 2025.07.14_cmake-for-cat branch 8 times, most recently from 57ec74c to d479a53 Compare July 22, 2025 14:05
@Treece-Burgess
Copy link
Contributor

@Treece-Burgess I am writing this message just to document that we will need to update the CI tests to account for the new directory structure introduced in this PR.

Yes, once you are done making changes. I think I will update the CI through this PR as well. I will default to use the CMake option. Is that fine with you?

@dbarry9
Copy link
Contributor Author

dbarry9 commented Jul 22, 2025

@Treece-Burgess I am writing this message just to document that we will need to update the CI tests to account for the new directory structure introduced in this PR.

Yes, once you are done making changes. I think I will update the CI through this PR as well. I will default to use the CMake option. Is that fine with you?

Yes, that is good.

@dbarry9 dbarry9 force-pushed the 2025.07.14_cmake-for-cat branch 13 times, most recently from 1573dc6 to 1c5a409 Compare July 25, 2025 21:05
@dbarry9 dbarry9 force-pushed the 2025.07.14_cmake-for-cat branch 8 times, most recently from f225505 to 4f2acd6 Compare July 28, 2025 16:12
@dbarry9 dbarry9 force-pushed the 2025.07.14_cmake-for-cat branch from 4f2acd6 to 7424ef5 Compare July 28, 2025 17:32
To accommodate different instruction sets and precisions available in
modern architectures, these changes add CMake as a more flexible build
system for the Counter Analysis Toolkit.

These changes have been tested on the Intel Sapphire Rapids
architecture.
@dbarry9 dbarry9 force-pushed the 2025.07.14_cmake-for-cat branch from 7424ef5 to 888cf82 Compare July 28, 2025 18:49
@Treece-Burgess Treece-Burgess added counter-analysis-toolkit PRs and Issues related to the counter analaysis toolkit status-work-in-progress PR is still being worked on labels Aug 14, 2025
@Treece-Burgess
Copy link
Contributor

During a previous PICL meeting is was discuss to use meson-python instead of autotools. Due to this, I discussed with @dbarry9 if continuing to keep this PR open was applicable. We both agreed that the CAT build system should be inline with the PAPI build system. Therefore, I will be closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

counter-analysis-toolkit PRs and Issues related to the counter analaysis toolkit status-work-in-progress PR is still being worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants