-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
add nvtx equivalent for rocm #940
base: develop
Are you sure you want to change the base?
Conversation
120e750
to
047d2f1
Compare
384d868
to
2f13436
Compare
build tested locally (cuda12) and (rocm 5.7.1) with |
7d1c6c7
to
d931ab6
Compare
5ad61e6
to
7e32349
Compare
7e32349
to
9a76008
Compare
317a941
to
e1f72d6
Compare
@simonpintarelli would it be possible to bring this up to date and possibly merge? |
e1f72d6
to
5c6ace8
Compare
Thanks for the reminder @gsavva. Is it correct that it worked for you on lumi? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to be merged
ping @gsavva |
- add dependency on `roctracer-dev` when(+rocm+nvtx) - conflict +nvtx when neither rocm nor cuda is enabled
5c6ace8
to
169ac14
Compare
The
On the other hand,
I would suggest merging this PR and investigate this issue separately (it might be related to the recent update of ROCm) |
it is indeed crashing, here is the backtrace:
|
Adding
https://github.com/ROCm/roctracer/tree/amd-master?tab=readme-ov-file#roctx-api
which is identical to the currently used nvtx api.
FindRocTX.cmake
tocmake/modules
+nvtx
isn't part of the ci