This is an archive of the discontinued LLVM Phabricator instance.

[Fuchsia] Omit the list of projects and runtimes
ClosedPublic

Authored by phosek on Apr 6 2022, 11:36 PM.

Details

Summary

The list of projects and runtimes is part of cache now. This addresses the
current failure which is due to a mismatch between the configuration and
the content of the cache file.

In addition, also include bolt in the list of projects that trigger our builders
since bolt is already enabled in the Fuchsia CMake cache file.

Diff Detail

Event Timeline

phosek created this revision.Apr 6 2022, 11:36 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2022, 11:36 PM
Herald added a subscriber: abrachet. · View Herald Transcript
phosek requested review of this revision.Apr 6 2022, 11:36 PM
rovka added inline comments.Apr 7 2022, 1:43 AM
zorg/buildbot/builders/FuchsiaBuilder.py
32

Bolt wasn't enabled before. You should at least mention it in the commit message, or even send a separate patch.

96

I think the runtimes will be handled differently than before, unless you specifically set enable_runtimes to 'auto'. AFAICT the default is None, which means the runtimes will be built via LLVM_ENABLE_PROJECTS instead of LLVM_ENABLE_RUNTIMES, which may lead to subtle differences. If this is on purpose, you should document it in the commit message.

phosek retitled this revision from [Fuchsia] The list of projects and runtimes is part of cache now to [Fuchsia] Omit the list of projects and runtimes.Apr 22 2022, 11:29 PM
phosek edited the summary of this revision. (Show Details)
phosek marked 2 inline comments as done.
phosek added inline comments.
zorg/buildbot/builders/FuchsiaBuilder.py
96

This builder uses CMake cache file which already sets these values, see https://github.com/llvm/llvm-project/blob/9b8a04313f0ec9b262c4c3c5182dc5566f53fe67/clang/cmake/caches/Fuchsia-stage2.cmake#L7. I have updated the comment to be more explicit.

gkistanova accepted this revision.Apr 23 2022, 12:15 AM

LGTM with a nit pick.

The list of projects and runtimes is part of cache now.

How about something like this instead?

The list of projects and runtimes is a part of Fuchsia-stage2.cmake <and whatever else cmake files> cache now.
This revision is now accepted and ready to land.Apr 23 2022, 12:15 AM
This revision was automatically updated to reflect the committed changes.
phosek marked an inline comment as done.