[PATCH] D126506: workflows: Use new custom runners from github

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 7 23:47:44 PDT 2022


tstellar added inline comments.


================
Comment at: .github/workflows/llvm-project-tests.yml:39-45
       - name: Setup Windows
         if: startsWith(matrix.os, 'windows')
         uses: llvm/actions/setup-windows at main
         with:
           arch: amd64
       - name: Install Ninja
         uses: llvm/actions/install-ninja at main
----------------
alexbatashev wrote:
> tstellar wrote:
> > alexbatashev wrote:
> > > Did you consider using sccache here? I see that right now this job takes ~4 hours, and the majority of time is spent on builds, which correlates with my experience building llvm in github actions. Enabling sccache should bring that down to ~3-15 minutes (windows is still much slower than linux, but that's a big improvement over a few hours).
> > Is sccache something that is integrated with GitHub or do we have to set up our own server somewhere?
> > 
> > With these new runners, most jobs take around 30 minutes which is a big improvement over the old runners.  Here is an example pull requests that uses the new runners: https://github.com/llvm/llvm-project-release-prs/pull/1
> That’s a fantastic improvement. But builds are still 39 minutes long. 
> 
> For github-hosted machines I’ve had success with github.com/actions/cache (see for example https://github.com/intel/llvm/commit/95104c079bacd4dd2920c8439d97d3f93e601b18). In my config llvm produced ~1,5 gigs of binaries. 10 gigs cache was more than enough for that task. 
> 
> On self-hosted machines that’s even easier. You can persist data on the machine itself: https://github.com/intel/llvm/blob/f6420c769d473c00be736b8df361e10dd98a6009/.github/workflows/sycl_windows_build_and_test.yml#L32
Thanks for this information.  I will look into enabling this in a follow-up patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126506/new/

https://reviews.llvm.org/D126506



More information about the llvm-commits mailing list