Skip to content

Commit

Permalink
[flang] Add builder/worker for debug build with reverse iteration
Browse files Browse the repository at this point in the history
For https://discourse.llvm.org/t/reverse-iteration-bots/72224.

This does not warrant a whole new builder, so I'm adding reverse
iteration to the debug configuration.

I thought of a few other places but:
* The release builder tests what we will later ship, so I don't
  want to change the configuration.
* Release with asserts is a very common developer configuration
  so it helps to know if problems occur with that only.
* The rest are build types, out of tree, shared libs, etc. which
  are niche enough I don't want reversed iterator problems polluting
  them.

Initially I'm not removing the existing worker, so that the two
can run in parallel with the reverse iterator bot on silent.

Once it's been setup and is stable, it'll move to normal and
I'll remove the old worker and builder.

Differential Revision: https://reviews.llvm.org/D157274
  • Loading branch information
DavidSpickett committed Aug 15, 2023
1 parent 0c5624e commit 809115c
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 0 deletions.
16 changes: 16 additions & 0 deletions buildbot/osuosl/master/config/builders.py
Original file line number Diff line number Diff line change
Expand Up @@ -1952,6 +1952,22 @@
"-DLLVM_USE_LINKER=lld",
])},

{'name' : "flang-aarch64-debug-reverse-iteration",
'tags' : ["flang", "rev_iter"],
'workernames' : ["linaro-flang-aarch64-debug-reverse-iteration"],
'builddir': "flang-aarch64-debug-reverse-iteration",
'factory' : UnifiedTreeBuilder.getCmakeWithNinjaBuildFactory(
clean=True,
checks=['check-flang'],
depends_on_projects=['llvm','mlir','clang','flang'],
extra_configure_args=[
"-DLLVM_TARGETS_TO_BUILD=AArch64",
"-DCMAKE_BUILD_TYPE=Debug",
"-DCMAKE_CXX_STANDARD=17",
"-DLLVM_USE_LINKER=lld",
"-DLLVM_REVERSE_ITERATION:BOOL=ON",
])},

{'name' : "flang-aarch64-latest-clang",
'tags' : ['flang'],
'workernames' : ["linaro-flang-aarch64-latest-clang"],
Expand Down
1 change: 1 addition & 0 deletions buildbot/osuosl/master/config/workers.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ def get_all():
create_worker("linaro-flang-aarch64-sharedlibs", properties={'jobs' : 32}, max_builds=1),
create_worker("linaro-flang-aarch64-out-of-tree", properties={'jobs' : 32}, max_builds=1),
create_worker("linaro-flang-aarch64-debug", properties={'jobs' : 32}, max_builds=1),
create_worker("linaro-flang-aarch64-debug-reverse-iteration", properties={'jobs' : 32}, max_builds=1),
create_worker("linaro-flang-aarch64-latest-clang", properties={'jobs' : 32}, max_builds=1),
create_worker("linaro-flang-aarch64-release", properties={'jobs' : 32}, max_builds=1),
create_worker("linaro-flang-aarch64-rel-assert", properties={'jobs' : 32}, max_builds=1),
Expand Down

0 comments on commit 809115c

Please sign in to comment.