Repository navigation
[perf experiment] remove niche from SyntaxContext - #164803
panstromek wants to merge 2 commits into
Conversation
|
@bors try @rust-timer queue benchmarks=ctfe-stress-5,deep-vector,issue-58319,large-workspace,libc-0.2.172,tt-muncher priority=-11 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
[perf experiment] remove niche from SyntaxContext
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (1473cc4): comparison URL. Overall result: no relevant changes - no action neededBenchmarking means the PR may be perf-sensitive. Consider adding rollup=never if this change is not fit for rolling up. @rustbot label: -S-waiting-on-perf -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (secondary -2.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesThis perf run didn't have relevant results for this metric. Binary sizeResults (primary -0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: missing data |
|
ok, let's try to remove the niche completely ( @bors try @rust-timer queue benchmarks=ctfe-stress-5,deep-vector,issue-58319,large-workspace,libc-0.2.172,tt-muncher priority=-11 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
[perf experiment] remove niche from SyntaxContext
|
The job Click to see the possible cause of the failure (guessed by this bot) |
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (4612f1e): comparison URL. Overall result: ✅ improvements - no action neededBenchmarking means the PR may be perf-sensitive. Consider adding rollup=never if this change is not fit for rolling up. @rustbot label: -S-waiting-on-perf -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (secondary 1.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesThis perf run didn't have relevant results for this metric. Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: missing data |
To test whether this causes the unexpected improvement in #164425
We could also test each of those newtypes separately, but I don't think it's super important, I just want to test the hypothesis.