- 
                Notifications
    You must be signed in to change notification settings 
- Fork 79
Allow opt-out of implicit bounds-checking #563
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
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.