Correct inlinealloc

[wsmoses]

No description provided.

[maleadt]

Thanks! I take it you want this backported to a version that doesn't require LLVM 6?

[wsmoses]

Yeah

[wsmoses]

I've also failed to actually make a test of this, but it does fix the bug we saw (see below):

julia> using ElectrochemicalKinetics
using GPUCompiler

julia> using GPUCompiler

# with the fix above
julia> GPUCompiler.is_inlinealloc(MarcusHushChidseyDOS{Float64})
false

[codecov]

Codecov Report

Patch coverage: 71.42% and project coverage change: -7.79 ⚠️

Comparison is base (c0c9ab7) 82.28% compared to head (7553ebf) 74.50%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #484      +/-   ##
==========================================
- Coverage   82.28%   74.50%   -7.79%     
==========================================
  Files          23       23              
  Lines        3082     3051      -31     
==========================================
- Hits         2536     2273     -263     
- Misses        546      778     +232     

Impacted Files	Coverage Δ
src/irgen.jl	81.31% <71.42%> (-12.08%)	⬇️

... and 10 files with indirect coverage changes

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

[wsmoses]

@maleadt could we also get a version tag for the LLVM6 compatible branch as well?