Hi. I tried the instructions in https://github.com/rust-lang/measureme/blob/master/summarize/Readme.md#profiling-the-nightly-compiler in Servo and got
Error: "Event stream file format version \'3\' is not supported\n by this version of `measureme`.". Is this a known issue?
Note that we’re on nightly-2019-12-23. Is a more recent Nightly required?
you're getting this when running summarize right ?
(I wonder if summarize can read old formats; just in case it's the tool which is too recent and not the nightly too old; rustc still uses measureme 0.5 and it seems the repo has a newer version; I think the version format in the repo is 5)
Yes, when running
summarize, which I compiled per instructions from
$ git clone https://github.com/rust-lang/measureme.git
@Simon Sapin can you try rebuilding measureme at the 0.5 or 0.5.1 tag?
yeah I suggested that elsewhere, and am also trying it rn
git checkout fb50acb will fix the format error yeah; I'm still seeing other "unexpectedly encountered events" but I'm on a newer nightly, I'll check on 12/23 next
yup it'll be enough, it's good on 12/23
0.5.1 works, thanks
@Wesley Wiser @mw was wondering, do y'all know what's going on here? It looks like measureme is updated to 0.7.1 in the rust-lang/rust repository, but is it still emitting an older format?
it's https://github.com/rust-lang/rust/pull/67397#issuecomment-573153365 IIUC — perf.rlo was apparently updated to the new summarize version already but it seems not to work yet
(0.7.1 emits version 5, if that's what you're talking about)
are you getting an error about version 3 ?
@mw 's PR that updates rustc to the latest version just merged today so this should be resolved in the next nightly.
@Alex Crichton ^^
(my understanding is this: if you have version 3, on nightly, use summarize 0.5.1;if you have version 5, on master, use summarize 0.7.1)
aha, nevermind, carry on then!
sorry for the bother :)