rustc-perf icon indicating copy to clipboard operation
rustc-perf copied to clipboard

Re-enable local std?

Open Mark-Simulacrum opened this issue 5 years ago • 3 comments

This tracks the potential revert of 048360c77a74244ba6e70d9b3f2bcd7779, which caused a 10% regression on the collection server.

Before we revert I would like to determine the cause of the regression, i.e., what difference it made that std was built on the collector rather than in Rust's CI. Ideally, that investigation will mean that we adjust Rust's CI to build a better std.

Mark-Simulacrum avatar Jul 27 '20 01:07 Mark-Simulacrum

Any progress here?

nnethercote avatar Aug 04 '20 23:08 nnethercote

@Mark-Simulacrum what's the latest on this?

rylev avatar Jul 29 '21 14:07 rylev

I don't recall doing any further investigation. I'm not sure if it's worth doing that still or if we should just close this.

I'd guess that we could try re-applying that commit ( 048360c) and see whether that gives us a performance win. If so, then exploring how we can adjust rustc mainline CI to get the same win may well be worthwhile.

Mark-Simulacrum avatar Jul 29 '21 15:07 Mark-Simulacrum