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

Only warn about missed transformations when debug output is enabled. #521

Merged
merged 1 commit into from
Sep 21, 2023

Conversation

maleadt
Copy link
Member

@maleadt maleadt commented Sep 21, 2023

julia> function main()
           function kernel(a)
               @loopinfo unroll for i = eachindex(a)
                   a[i] = 0
               end
               return
           end

           arr = CUDA.zeros(10)
           @cuda kernel(arr)
       end
main (generic function with 1 method)

julia> empty!(CUDA.compiler_cache(context())); main()
CUDA.HostKernel for kernel(CuDeviceVector{Float32, 1})

julia> ENV["JULIA_DEBUG"] = "GPUCompiler"
"GPUCompiler"

julia> empty!(CUDA.compiler_cache(context())); main()
warning: abstractarray.jl:699:0: loop not unrolled: the optimizer was unable to perform the requested transformation; the transformation might be disabled or specified as part of an unsupported transformation ordering
CUDA.HostKernel for kernel(CuDeviceVector{Float32, 1})

cc @pchintalapudi

@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Patch coverage: 66.66% and project coverage change: +0.82% 🎉

Comparison is base (edfdc1a) 83.18% compared to head (ce900c4) 84.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #521      +/-   ##
==========================================
+ Coverage   83.18%   84.00%   +0.82%     
==========================================
  Files          24       24              
  Lines        3300     3302       +2     
==========================================
+ Hits         2745     2774      +29     
+ Misses        555      528      -27     
Files Changed Coverage Δ
src/optim.jl 88.51% <50.00%> (-0.24%) ⬇️
src/utils.jl 94.00% <100.00%> (+0.12%) ⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@maleadt maleadt merged commit d84a484 into master Sep 21, 2023
@maleadt maleadt deleted the tb/missed_transforms branch September 21, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant