This is an archive of the discontinued LLVM Phabricator instance.

[AIX][PowerPC] Change AIX buildbot to target native AIX build
ClosedPublic

Authored by ZarkoCA on Dec 7 2021, 4:29 PM.

Details

Summary

The AIX buildbot has been setting as ppc64le as its target. Change it to target
AIX instead.

Diff Detail

Event Timeline

ZarkoCA created this revision.Dec 7 2021, 4:29 PM
ZarkoCA requested review of this revision.Dec 7 2021, 4:29 PM
daltenty accepted this revision.Dec 9 2021, 6:40 AM
daltenty added inline comments.
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?

This revision is now accepted and ready to land.Dec 9 2021, 6:40 AM
ZarkoCA added inline comments.Dec 9 2021, 6:48 AM
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?

daltenty added inline comments.Dec 9 2021, 11:44 AM
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'}
ZarkoCA updated this revision to Diff 393446.Dec 10 2021, 4:22 AM
  • Add OBJECT_MODE
ZarkoCA marked 2 inline comments as done.Dec 10 2021, 4:22 AM
ZarkoCA added inline comments.
buildbot/osuosl/master/config/builders.py
667

Got it, did that here as well. Thanks.

daltenty accepted this revision.Dec 10 2021, 12:31 PM

LGTM, thanks!

This revision was automatically updated to reflect the committed changes.
ZarkoCA marked an inline comment as done.

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.

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.

Yes, I did forget. Thank you.