-
Notifications
You must be signed in to change notification settings - Fork 70
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
Allow opt-out of implicit bounds-checking #563
Allow opt-out of implicit bounds-checking #563
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## vc/pocl #563 +/- ##
========================================
Coverage 0.00% 0.00%
========================================
Files 12 21 +9
Lines 777 1513 +736
========================================
- Misses 777 1513 +736 ☔ View full report in Codecov by Sentry. |
Benchmark Results
Benchmark PlotsA plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR. |
48e3752
to
e565304
Compare
e565304
to
4dd0acc
Compare
777c099
to
3bb80ac
Compare
4dd0acc
to
5e03ecf
Compare
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.
Some suggestions could not be made:
- src/KernelAbstractions.jl
- lines 97-98
5e03ecf
to
e0c44ee
Compare
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.
Some suggestions could not be made:
- src/KernelAbstractions.jl
- lines 97-98
e0c44ee
to
a73ea1f
Compare
a73ea1f
to
ed45e9b
Compare
KernelAbstractions currently creates kernels that look like: ``` if __validindex(ctx) # Body end ``` This is problematic due to the convergence requirement on `@synchronize`.
ed45e9b
to
0aa6a5b
Compare
KernelAbstractions currently creates kernels that look like:
This is problematic due to the convergence requirement on
@synchronize
.