search

JuliaLabs / Cassette.jl

Overdub Your Julia Code
Other
370 stars 34 forks source link

set `ssaflags` #194

Closed aviatesk closed 3 years ago

aviatesk commented 3 years ago

On recent master, (code_info::CodeInfo).ssaflags might be set before inference and observed during inference, and the overdubbing mechanism should modify it so that its length matches the length of statements (otherwise inference code throws).

Ideally we should respect flags that are originally set though.

codecov-commenter commented 3 years ago

Codecov Report

Merging #194 (07d4db3) into master (224659a) will increase coverage by 0.07%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #194      +/-   ##
==========================================
+ Coverage   85.05%   85.12%   +0.07%     
==========================================
  Files           6        6              
  Lines         756      780      +24     
==========================================
+ Hits          643      664      +21     
- Misses        113      116       +3
Impacted Files Coverage Δ
src/overdub.jl 95.84% <100.00%> (+0.16%) :arrow_up:
src/tagging.jl 72.32% <0.00%> (-0.33%) :arrow_down:
src/pass.jl 100.00% <0.00%> (ø)

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 224659a...07d4db3. Read the comment docs.