Skip to content
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

Update GPUCompiler to fix unreachable control flow. #1951

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

maleadt
Copy link
Member

@maleadt maleadt commented Jun 12, 2023

Adapts to/requires JuliaGPU/GPUCompiler.jl#467, fixes #1746.

@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Patch coverage: 66.66% and project coverage change: -0.02 ⚠️

Comparison is base (fc8f7e4) 62.02% compared to head (369c6e8) 62.01%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1951      +/-   ##
==========================================
- Coverage   62.02%   62.01%   -0.02%     
==========================================
  Files         150      150              
  Lines       12510    12506       -4     
==========================================
- Hits         7759     7755       -4     
  Misses       4751     4751              
Impacted Files Coverage Δ
lib/cudadrv/state.jl 78.68% <50.00%> (-0.32%) ⬇️
src/compiler/compilation.jl 88.03% <100.00%> (+0.22%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@maleadt
Copy link
Member Author

maleadt commented Jun 13, 2023

Looks like 11.4 isn't sufficient as a lower-bound, as there's different failures until I use 11.5.

@maleadt maleadt added the cuda kernels Stuff about writing CUDA kernels. label Jun 13, 2023
@maleadt maleadt merged commit 1ef29ec into master Jun 14, 2023
@maleadt maleadt deleted the tb/lower_unreachable branch June 14, 2023 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda kernels Stuff about writing CUDA kernels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unreachable control flow leads to illegal divergent barriers
1 participant