-
Notifications
You must be signed in to change notification settings - Fork 37
[TLE-Raw] implement vassert and add unit test #325
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
Conversation
sunnycase
left a comment
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.
LGTM
sgjzfzzf
left a comment
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.
Nice done! Btw could u please apply the newly added operation to topk.py to check the bdimx to be 1024 during runtime? That would be very helpful.
zhzhcookie
left a comment
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.
LGTM
This PR implements the vassert function for the TLE Raw DSL.
Changes:
Implemented vassert in utils.py by targeting the correct __assertfail symbol, enabling native CUDA device-side assertions.
Added a new test case python/tutorials/tle/raw/05-test-assert.py to verify that assertions correctly terminate the kernel and report errors.