Skip to content

Latest commit

 

History

History
87 lines (70 loc) · 4.15 KB

updating_clang.md

File metadata and controls

87 lines (70 loc) · 4.15 KB
Raw

Updating clang

We distribute prebuilt packages of LLVM binaries, including clang and lld, that all developers and bots pull at gclient runhooks time. These binaries are just regular LLVM binaries built at a fixed upstream revision. This document describes how to build a package at a newer revision and update Chromium to it. An archive of all packages built so far is at https://is.gd/chromeclang

  1. Check that https://ci.chromium.org/p/chromium/g/chromium.clang/console looks reasonably green. Red bots with seemingly normal test failures are usually ok, that likely means the test is broken with the stable Clang as well.
  2. Sync your Chromium tree to the latest revision to pick up any plugin changes.
  3. Run go/chrome-push-clang-to-goma. This takes a recent dry run CL to update clang, and if the trybots were successful it will copy the binaries from the staging bucket to the production one. Writing to this bucket requires special permissions. File a bug at g.co/bugatrooper if you don't have these already (e.g., https://crbug.com/1034081). Then it will push the packages to RBE. If you do not have the necessary credentials to do the upload, ask [email protected] to find someone who does.
    • Alternatively, to create your own roll CL, you can manually run tools/clang/scripts/upload_revision.py with a recent upstream LLVM commit hash as the argument. After the *_upload_clang trybots are successfully finished, run go/chrome-promote-clang on the new Clang package name.
  4. Run tools/clang/scripts/sync_deps.py to update the deps entries in DEPS.
  5. Run an exhaustive set of try jobs to test the new compiler. The CL description created previously by upload_revision.py includes Cq-Include-Trybots: lines for all needed bots, so it's sufficient to just run git cl try (or hit "CQ DRY RUN" on gerrit).
  6. Commit the roll CL from the previous step.
  7. The bots will now pull the prebuilt binary, and RBE will have a matching binary, too.

Performance regressions

After doing a clang roll, you may get a performance bug assigned to you (example). Some performance noise is expected while doing a clang roll.

You can check all performance data for a clang roll via https://chromeperf.appspot.com/group_report?rev=XXXXXX, where XXXXXX is the revision number, e.g. 778090 for the example bug (look in the first message of the performance bug to find this). Click the checkboxes to display graphs. Hover over points in the graph to see the value and error.

Serious regressions require bisecting upstream commits (TODO: how to repro?). If the regressions look insignificant and there is green as well as red, you can close the bug as "WontFix" with an explanation.

Adding files to the clang package

The clang package is downloaded unconditionally by all bots and devs. It's called "clang" for historical reasons, but nowadays also contains other mission-critical toolchain pieces besides clang.

We try to limit the contents of the clang package. They should meet these criteria:

  • things that are used by most developers use most of the time (e.g. a compiler, a linker, sanitizer runtimes)
  • things needed for doing official builds

Adding a New Package

If you want to make artifacts available that do not meet the criteria for being included in the "clang" package, you can make package.py upload it to a separate zip file and then download it on an opt-in basis by using update.py's --package option. Here is [an example of adding a new package].

To test changes to package.py, change CLANG_SUB_REVISION in update.py to a random number above 2000 and run the *_upload_clang trybots.

Once the change to package.py is in, file a bug under Tools > LLVM requesting that a new package be created ([example bug]).

Once it's been uploaded and rolled, you can download it via:

tools/clang/scripts/update.py --package your-package-name

an example of adding a new package [example bug]: https://crbug.com/335730441