JuliaLabs / MLIR.jl

MIT License
55 stars 8 forks source link

Update toy brutus #34

Closed Pangoraw closed 8 months ago

Pangoraw commented 8 months ago

Needs #33

Pangoraw commented 8 months ago

CI on 1.9

 LLVM_full_jll [a3ccf953] log:
 ├─possible versions are: 8.0.1-17.0.6 or uninstalled
 ├─restricted to versions 15 by an explicit requirement, leaving only versions: 15.0.7
 └─restricted by julia compatibility requirements to versions: 8.0.1-14.0.6 or uninstalled — no versions left

edit: this is fixed by #27

codecov-commenter commented 8 months ago

Codecov Report

Attention: 4087 lines in your changes are missing coverage. Please review.

Comparison is base (d239921) 41.04% compared to head (b3fd104) 1.90%. Report is 19 commits behind head on main.

Files Patch % Lines
src/Dialects/14/Tosa.jl 0.00% 501 Missing :warning:
src/Dialects/14/Arithmetic.jl 0.00% 322 Missing :warning:
src/Dialects/14/Shape.jl 0.00% 277 Missing :warning:
src/Dialects/14/PDLInterp.jl 0.00% 276 Missing :warning:
src/Dialects/14/Vector.jl 0.00% 271 Missing :warning:
src/Dialects/14/GPU.jl 0.00% 232 Missing :warning:
src/Dialects/14/MemRef.jl 0.00% 206 Missing :warning:
src/Dialects/14/Async.jl 0.00% 193 Missing :warning:
src/Dialects/14/Math.jl 0.00% 192 Missing :warning:
src/Dialects/14/OpenMP.jl 0.00% 192 Missing :warning:
... and 19 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #34 +/- ## ========================================== - Coverage 41.04% 1.90% -39.14% ========================================== Files 6 107 +101 Lines 614 26054 +25440 ========================================== + Hits 252 496 +244 - Misses 362 25558 +25196 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Pangoraw commented 8 months ago
# Error @ /home/runner/work/MLIR.jl/MLIR.jl/src/Dialects/15/ArmNeon.jl:46:10
"""
function 2d_sdot(a::Value, b::Value, c::Value; res::MLIRType, location=Location())
#        ╙ ── Invalid signature in function definition
vchuravy commented 8 months ago

I am going to merge this despite the failure on nightly.