Conversation
This updates LLVM WoA bots with following changes: 1) Do not run stage1 testing on 2 stage clang bot. 2) Remove redundant workers. 3) Remove -DCOMPILER_RT_BUILD_PROFILE=OFF 4) Add compiler-rt to single stage bot 5) Set clean=False for the 2stage bots 6) Set "-DCLANG_DEFAULT_LINKER=lld" for all WoA bots
What is the reason to not add this onto the 2 stage bots? I can think of a few:
Any more? Assuming we have the hardware resource I agree with the decision. |
| "-DMLIR_RUN_ARM_SVE_TESTS=True", | ||
| "-DLLVM_LIT_ARGS='-v'"])}, | ||
|
|
||
| {'name' : "clang-arm64-windows-msvc-testsuite", |
There was a problem hiding this comment.
I know this is very late to say this, but it is confusing that we have "msvc" in the bot names. Could we change that for this new one? clang-arm64-windows-testsuite would be fine.
A couple of times I've had to explain that the "msvc" bot actually uses clang-cl.
I would have proposed changing all the names but that's a fiddly multi week process so I never had the energy to do it.
| checkout_lld=True, | ||
| runTestSuite=True, | ||
| testWithLNT=False, | ||
| testsuite_flags=["-DTEST_SUITE_SUBDIRS='Fortran'"], |
There was a problem hiding this comment.
Does enabling flang not just automatically do this?
| # so, before that's fixed, disable everything that triggers its build. | ||
| "-DCOMPILER_RT_BUILD_SANITIZERS=OFF", | ||
| "-DCOMPILER_RT_BUILD_PROFILE=OFF"])}, | ||
| "-DCOMPILER_RT_BUILD_SANITIZERS=OFF"])}, |
There was a problem hiding this comment.
On Windows on Arm, we are only building compiler-rt for the compiler runtime part, correct? I see that all these are disabling the sanitizers.
| create_worker("linaro-g3-04", max_builds=1), | ||
|
|
||
| # AArch64 Windows Microsoft Surface X Pro | ||
| create_worker("linaro-armv8-windows-msvc-01", max_builds=1), |
There was a problem hiding this comment.
Be careful with the timing of when this change lands vs. when you remove the agent from the actual machine. I managed to get our server IP banned because I left some workers running that kept trying to connect to the build master.
If the machine is offline before this lands there should be no problem.
| 'builddir': "clang-arm64-windows-msvc-testsuite", | ||
| 'factory' : ClangBuilder.getClangCMakeBuildFactory( | ||
| vs="manual", | ||
| checks=[], |
There was a problem hiding this comment.
Where are you disabling the check stage 1 and 2 here? Also this doesn't seem to be a 2 stage, there's no useTwoStage.
2 participants
This is a follow up patch on #245 and #252
It adds a new builder called clang-arm64-windows-msvc-testsuite. It will
do stage1 and stage2 builds without running checks. Checks are already
covered by single stage and 2 stage bots.