This is an archive of the discontinued LLVM Phabricator instance.

[clang-ppc64-aix] Switch build compiler to clang
ClosedPublic

Authored by Jake-Egan on Aug 11 2022, 10:32 AM.

Details

Summary

When LLVM updated the C++ standards to 17, the AIX bot encountered multiple related build issues because ibm-clang doesn't currently have a release to support C++17. So, this patch switches the build compiler for AIX from ibm-clang to clang as a step to mitigate that.

Event Timeline

Jake-Egan created this revision.Aug 11 2022, 10:32 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: shchenz. · View Herald Transcript
Jake-Egan requested review of this revision.Aug 11 2022, 10:32 AM
Jake-Egan edited the summary of this revision. (Show Details)Aug 11 2022, 10:41 AM
daltenty added inline comments.Aug 11 2022, 11:08 AM
buildbot/osuosl/master/config/builders.py
697–717

Maybe it's better to not include the specific release candidate in the path here? That will give more flexibility to update to the final release once it's available without necessitating yet another change.

Removed release candidate from path.

  • [clang-ppc64-aix] Switch build compiler to clang

Hopefully fix bad arc diff.

Jake-Egan marked an inline comment as done.Aug 11 2022, 12:05 PM
buildbot/osuosl/master/config/builders.py
697–717

Can we check the installation path conventions for AIX deployments? I would think this is /usr/local territory.

Jake-Egan updated this revision to Diff 451965.Aug 11 2022, 1:29 PM

Change path to /usr/local/.

This revision is now accepted and ready to land.Aug 11 2022, 1:44 PM
This revision was automatically updated to reflect the committed changes.
zorg/jenkins/jobs/jobs/lldb-cmake-matrix