Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NVTX Macros #276

Merged
merged 27 commits into from
Oct 6, 2022
Merged

NVTX Macros #276

merged 27 commits into from
Oct 6, 2022

Conversation

tylera-nvidia
Copy link
Collaborator

Create a custom set of macros for NVTX flags in MatX. Implement an initial set throughout the code and provide examples of how to use them.

@tylera-nvidia tylera-nvidia marked this pull request as ready for review October 3, 2022 21:54
@tylera-nvidia tylera-nvidia requested review from cliffburdick and luitjens and removed request for cliffburdick October 3, 2022 21:54
@tylera-nvidia
Copy link
Collaborator Author

[----------] Global test environment tear-down
[==========] 1095 tests from 327 test suites ran. (245252 ms total)
[ PASSED ] 1095 tests.

YOU HAVE 44 DISABLED TESTS

docker@ipp1-1946:/home/tallen/scratch/MatX2/build$ git rev-parse HEAD
f6dee29
docker@ipp1-1946:/home/tallen/scratch/MatX2/build$

Copy link
Collaborator

@cliffburdick cliffburdick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need a unit test showing how the user can do the USER level ones?

bench/00_transform/conv.cu Show resolved Hide resolved
docs_input/api/nvtx.rst Outdated Show resolved Hide resolved
include/matx/core/nvtx.h Show resolved Hide resolved
@tylera-nvidia tylera-nvidia merged commit e5de7bd into main Oct 6, 2022
@tylera-nvidia tylera-nvidia deleted the autoNVTX branch October 11, 2022 15:56
cliffburdick pushed a commit that referenced this pull request Dec 8, 2023
NVTX Macros added for automated NVTX Ranges for the MatX API. enable with MATX_NVTX_FLAGS=ON with cmake build.
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.

3 participants