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

eliminates hacky overloads ! #129

Merged
merged 4 commits into from
Mar 11, 2021
Merged

eliminates hacky overloads ! #129

merged 4 commits into from
Mar 11, 2021

Conversation

aviatesk
Copy link
Owner

JuliaLang/julia#39305 is merged, and now we can
really clean up our abstractinterpretation.jl.
The legacy hacky overloads will be kept in
src/legacy/abstractinterpretation.jl
for the compatbility with Julia v1.6.

JET benchmark will also show the impact analysis of constant prop' on
union-split signatures on JET analysis.

JuliaLang/julia#39305 is merged, and now we can
really clean up our `abstractinterpretation.jl`.
The legacy hacky overloads will be kept in 
`src/legacy/abstractinterpretation.jl`
for the compatbility with Julia v1.6.

JET benchmark will also show the impact analysis of constant prop' on
union-split signatures on JET analysis.
@aviatesk aviatesk closed this Mar 11, 2021
@aviatesk aviatesk reopened this Mar 11, 2021
@codecov-io
Copy link

codecov-io commented Mar 11, 2021

Codecov Report

Merging #129 (561a3e4) into master (b35d569) will decrease coverage by 0.02%.
The diff coverage is 86.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
- Coverage   82.47%   82.44%   -0.03%     
==========================================
  Files          11       11              
  Lines        1164     1185      +21     
==========================================
+ Hits          960      977      +17     
- Misses        204      208       +4     
Impacted Files Coverage Δ
src/abstractinterpreterinterface.jl 61.90% <ø> (ø)
src/abstractinterpretation.jl 96.38% <86.11%> (-1.62%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b35d569...561a3e4. Read the comment docs.

@aviatesk aviatesk merged commit e49cc95 into master Mar 11, 2021
@aviatesk aviatesk deleted the cleaninterface branch March 11, 2021 11:15
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.

2 participants