Commit c5b1a1c2 authored by Adrian McCarthy's avatar Adrian McCarthy Committed by Commit Bot

Update clang sheriff documentation

The `got_clang_revision` property is no longer available, so point out
that the LLVM revision range is available in the `gclient runhooks` log.

Change-Id: I5adde8dfc5e0a9ee6da0b852803744497d4c240a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2537700
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#831007}
parent 03ab462d
...@@ -91,8 +91,13 @@ and what to do about them: ...@@ -91,8 +91,13 @@ and what to do about them:
This is probably the most common bug. The standard procedure is to do these This is probably the most common bug. The standard procedure is to do these
things: things:
1. Use `got_clang_revision` property from first red and last green build to find 1. Open the `gclient runhooks` stdout log from the first red build. Near the
upstream regression range top of that log you can find the range of upstream llvm revisions. For
example:
From https://github.com/llvm/llvm-project
f917356f9ce..292e898c16d master -> origin/master
1. File a crbug documenting the crash. Include the range, and any other bots 1. File a crbug documenting the crash. Include the range, and any other bots
displaying the same symptoms. displaying the same symptoms.
1. All clang crashes on the Chromium bots are automatically uploaded to 1. All clang crashes on the Chromium bots are automatically uploaded to
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment