The AIX buildbot has been setting as ppc64le as its target. Change it to target
AIX instead.
Details
Diff Detail
- Repository
- rZORG LLVM Github Zorg
Event Timeline
buildbot/osuosl/master/config/builders.py | ||
---|---|---|
667 | Out of curiosity, do we set OBJECT_MODE somewhere here or rely on the worker side to set it to make sure we get the 64-bit variant? |
buildbot/osuosl/master/config/builders.py | ||
---|---|---|
667 | We are relying on the worker to set OBJECT_MODE=64. Would it better to hard code that as the target? |
buildbot/osuosl/master/config/builders.py | ||
---|---|---|
667 | Hardcoding it as a target won't work presay, since that just affects the default target. We actually need it to be part of the build env, so CMake configures for the right bitmode. For the libc++ buildbots we have it hardcoded in the env, so that there's no chance of the builder diverging from the specified bitmode based on the worker venv. We could probably do something similar here by passing an env to the build factory: ClangBuilder.getClangCMakeBuildFactory(env={'OBJECT_MODE': '64'} |
buildbot/osuosl/master/config/builders.py | ||
---|---|---|
667 | Got it, did that here as well. Thanks. |
Hi Zarko,
This patch has broken llvm-zorg.
There is no worker with the name "aix-ppc64" in the workers.py.
Did you forget to rename "aix-ppc64-ppc64le" to "aix-ppc64"?
If so, I have fixed that for you in https://github.com/llvm/llvm-zorg/commit/d4d8bac3d9d02ad73201dd2d5156724a7e421db0. Don't forget to rename you worker in the buildbot.tac file and restart.
Otherwise, you would need to propose a patch for workers.py.
Out of curiosity, do we set OBJECT_MODE somewhere here or rely on the worker side to set it to make sure we get the 64-bit variant?