search

timholy / SnoopCompile.jl

Provide insights about latency (TTFX) for Julia packages
https://timholy.github.io/SnoopCompile.jl/dev/
Other
309 stars 48 forks source link

Update for julia#43990 #279

Closed timholy closed 2 years ago

timholy commented 2 years ago

This updates both code and tests for the precompilation changes in https://github.com/JuliaLang/julia/pull/43990. Most urgently, a new invalidation location tag, "jl_insert_method_instance", was added, and analysis will error unless fixed. There are also subtle changes in attribution that require small changes elsewhere.

codecov[bot] commented 2 years ago

Codecov Report

Merging #279 (f9171bf) into master (66a23ef) will increase coverage by 4.20%. The diff coverage is 20.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
+ Coverage   82.58%   86.78%   +4.20%     
==========================================
  Files          16       16              
  Lines        1912     2028     +116     
==========================================
+ Hits         1579     1760     +181     
+ Misses        333      268      -65
Impacted Files Coverage Δ
src/invalidations.jl 89.25% <20.00%> (+2.21%) :arrow_up:
src/parcel_snoopc.jl 78.15% <0.00%> (-0.43%) :arrow_down:
src/visualizations.jl 0.00% <0.00%> (ø)
src/parcel_snoopi.jl 94.14% <0.00%> (+0.30%) :arrow_up:
SnoopCompileCore/src/snoopi_deep.jl 92.00% <0.00%> (+0.33%) :arrow_up:
src/parcel_snoopl.jl 90.00% <0.00%> (+0.52%) :arrow_up:
src/write.jl 92.30% <0.00%> (+1.00%) :arrow_up:
src/deep_demos.jl 58.97% <0.00%> (+1.07%) :arrow_up:
src/parcel_snoopi_deep.jl 89.77% <0.00%> (+1.78%) :arrow_up:
... and 1 more

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 66a23ef...f9171bf. Read the comment docs.