Kristoffer Carlsson

The AllocOptPass is potentially slow (and runs too often) in some cases

3

Looking a bit into why https://github.com/JuliaLang/julia/pull/54520 caused such a big latency improvement, the issue (or at least one issue) is that with the broadcast code a large majority of the...

`instantiate` always instantiates whole workspace

3

https://github.com/JuliaLang/Pkg.jl/blob/162634c5615d12b889e4b64f3cff95d1c377f189/src/API.jl#L1274-L1276 will download all packages + artfacts from packages in the manifest (which will be the whole workspace), not limited to the currently active project.

fix not hardcoding "Project.toml" when fixing up extensions in manifest

2

Fixes https://github.com/JuliaLang/Pkg.jl/issues/3815

Things get very confusing when putting the wrong name to a UUID in the project file

2

Try a project file with: ``` [deps] ForwardDiff = "0f1e0344-ec1d-5b48-a673-e5cf874b6c29" ``` activate it and then do: ``` julia> Pkg.add("ForwardDiff") Resolving package versions... Updating `/tmp/jl_RX2R13/TestWeakDepProject/Project.toml` [0f1e0344] + ForwardDiff v0.8.21 Updating `/tmp/jl_RX2R13/TestWeakDepProject/Manifest.toml`...

Remove `@test_nowarn` from tests

1

Failing the tests because of a single deprecation warning or anything getting emitted to stderr is very strong. This package fails tests, cf https://s3.amazonaws.com/julialang-reports/nanosoldier/pkgeval/by_hash/e8caa27_vs_f23fc0d/StochasticDelayDiffEq.1.7.0-rc2-73a7a8e46fe.log ``` ┌ Warning: dt

Understand why `let` wrapping the compile workload can lead to worse latency

2

This was observed in https://github.com/JuliaLang/PrecompileTools.jl/pull/16. It would be good to understand the interaction with `let` here since ideally, it would be good to keep it.

This is a reduction of a test in `test/strings/annotated.jl` that avoids loading StyledStrings (which now gets loaded when loading Test.jl): ```julia str1 = Base.AnnotatedString("test", [(1:4, :label => 5)]) str2 =...

``` Test Failed at /home/pkgeval/.julia/packages/LLVMLoopInfo/PhL9I/test/vectorization_tests.jl:73 Expression: numoccurences(r"call @llvm.masked.load.v8f64.p0v8f64", str) == 2 Evaluated: 0 == 2 ```

recommend against `using Foo` in package code (or "using considered harmful")

104

Cross-ref: https://github.com/JuliaLang/julia/pull/53428 --- I feel we are heading up against a "`using` crisis" where any new feature that is implemented by exporting a new name (either in Base or a...