Commit ae31f85
committed
[RISCV] Move clang-riscv-rva23-evl-vec-2stage to annotated builder
The intent is for the rest of the RISE RISC-V builders to follow, but
this starts the process.
Previously the builders used the framework provided by ClangBuilder.py.
As the build has required more customisation (with more to come - such
as building the llvm-test-suite with the stage1 compiler which isn't
support by ClangBuilder) this is requiring more and more changes to
increase configurability of ClangBuilder, but this makes the logic
harder to follow and I think we've reached the limit of what's
reasonable.
The AnnotatedBuilder infrastructure allows the build logic to just be
put in a shell script. The latest version of llvm-zorg is checked out
and the script is executed. This has the additional advantage of
decoupling script updates from needing to wait for buildmaster
redeploys. With that in mind, I've avoided putting any configuration
through options or environment variables defined by the builder
instantiation in builders.py so there's no need to reason about cases
where e.g. a script update was committed but the buildmaster may or may
not have been updated with a new builders.py.
rise-riscv-build.sh is written with the philosophy that "A little
copying is better than a little dependency." We could look to share more
logic, but we'd end up with the same configurability issue as
ClangBuilder. Instead, the script provides only the configurability
needed for this set of builders.
A future improvement would be to make rise-riscv-build.sh more friendly
to running outside of the buildbot environment (e.g. for local testing).
This is left to future work, as it's not a regression vs the current
approach.
This change has been tested locally along the lines of the instructions
I wrote in
<https://llvm.org/docs/HowToAddABuilder.html#testing-a-builder-config-locally>.
Something like the following is sufficient to use a local downstream
testing llvm-zorg branch for the annotated builder scripts temporarily:
```
--- a/zorg/buildbot/builders/AnnotatedBuilder.py
+++ b/zorg/buildbot/builders/AnnotatedBuilder.py
@@ -77,6 +77,8 @@ def getAnnotatedBuildFactory(
# Check out zorg so we can run the annotator scripts.
f.addGetSourcecodeForProject(
name='update-annotated-scripts',
+ repourl='file:///home/asb/llvm-zorg',
+ branch='synced-worktree',
project='zorg',
src_dir='llvm-zorg',
alwaysUseLatest=True)
```1 parent 3c627f0 commit ae31f85
File tree
2 files changed
+112
-40
lines changed- buildbot/osuosl/master/config
- zorg/buildbot/builders/annotated
2 files changed
+112
-40
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3362 | 3362 | | |
3363 | 3363 | | |
3364 | 3364 | | |
3365 | | - | |
3366 | | - | |
3367 | | - | |
3368 | | - | |
3369 | | - | |
3370 | | - | |
3371 | | - | |
3372 | | - | |
3373 | | - | |
3374 | | - | |
3375 | | - | |
3376 | | - | |
3377 | | - | |
3378 | | - | |
3379 | | - | |
3380 | | - | |
3381 | | - | |
3382 | | - | |
3383 | | - | |
3384 | | - | |
3385 | | - | |
3386 | | - | |
3387 | | - | |
3388 | | - | |
3389 | | - | |
3390 | | - | |
3391 | | - | |
3392 | | - | |
3393 | | - | |
3394 | | - | |
3395 | | - | |
3396 | | - | |
3397 | | - | |
3398 | | - | |
3399 | | - | |
3400 | | - | |
3401 | | - | |
3402 | | - | |
3403 | | - | |
3404 | | - | |
| 3365 | + | |
| 3366 | + | |
| 3367 | + | |
| 3368 | + | |
| 3369 | + | |
3405 | 3370 | | |
3406 | 3371 | | |
3407 | 3372 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
0 commit comments