-
Notifications
You must be signed in to change notification settings - Fork 90
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
[SYCLomatic] Add run length encode nontrivial runs API #1083
[SYCLomatic] Add run length encode nontrivial runs API #1083
Conversation
Signed-off-by: Matthew Michel <[email protected]>
Signed-off-by: Matthew Michel <[email protected]>
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.
I think this is improved, but I think there may be a bug and some unnecessary code.
Signed-off-by: Matthew Michel <[email protected]>
Signed-off-by: Matthew Michel <[email protected]>
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.
Thanks, LGTM.
Signed-off-by: Matthew Michel <[email protected]>
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
Signed-off-by: Matthew Michel <[email protected]>
Signed-off-by: Matthew Michel <[email protected]>
…de API (oneapi-src#393) Signed-off-by: Matthew Michel <[email protected]>
This PR implements the run length encoding nontrivial runs API. In order for the added source code in this PR to compile, oneapi-src/oneDPL#1019 must be first merged into oneDPL. The corresponding test PR for this implementation is provided in oneapi-src/SYCLomatic-test#393
Nontrivial runs are a subsequence of length two or greater in a buffer that are equivalent. This API marks the start of each nontrivial run, length of each nontrivial run, and the number of nontrivial runs.