jameysharp opened PR #6393 from jameysharp:guest-profiler-docs
to bytecodealliance:main
:
This isn't nearly as detailed as the other profiling chapters but I'm struggling to figure out what's important to say. I'd welcome feedback, although of course we can merge this and fill in details later.
This documentation reflects the changes merged in #6362, but that merged after the 9.0 branch was created. Perhaps we should backport that PR to 9.0 so the older CLI flags never appear in a release?
jameysharp requested pchickey for a review on PR #6393.
jameysharp requested wasmtime-default-reviewers for a review on PR #6393.
alexcrichton submitted PR review.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
Mind rewording the seconds around here so the list here can be a 3-bullet list with perf/vtune/guest profiling? I suspect many will scan over this page, see two bullets, and deduce there's two means of profiling Wasmtime. I'm sure many more yet will read further and realize there's a third, but having them all in one organized bulleted list I think makes sense (with all the existing commentary about which is appropriate of course)
fitzgen submitted PR review.
fitzgen submitted PR review.
fitzgen created PR review comment:
This should maybe mention that you can load the profile in perf.firefox.com? Would be nice to include a screenshot as well.
jameysharp updated PR #6393.
jameysharp created PR review comment:
It definitely should mention that. Also, it does mention it, on line 5. :grin:
A screenshot or two is a good idea but I'm not feeling up to doing that right this moment, so I'd like to merge without it.
jameysharp created PR review comment:
I think it's useful to have the distinction and separation between native and guest profilers, but I've added a bullet point to direct people to keep reading; do you think that takes care of it?
alexcrichton submitted PR review:
:+1:
jameysharp merged PR #6393.
fitzgen created PR review comment:
Lol whoops don’t know how I missed that
Last updated: Dec 23 2024 at 12:05 UTC