This is an archive of the discontinued LLVM Phabricator instance.

[test-release.sh] Add an option to build lib projects as a part of default runtimes.
AbandonedPublic

Authored by ldionne on Dec 4 2021, 1:37 PM.

Details

Reviewers
tstellar
amyk
Summary

This patch adds an option to the test-release.sh script to build some of the LLVM projects
(particularly compiler-rt/libcxx/libcxxabi/libunwind) as the default runtimes (with the -LLVM_ENABLE_RUNTIMES)
CMake invocation.

The intention behind this patch is not only that we plan to build some of these projects in this way
on AIX, but also is in correspondence to the thread that @ldionne previously posted regarding the changes
in how we build these projects at https://lists.llvm.org/pipermail/llvm-dev/2021-October/153238.html.

Diff Detail

Event Timeline

amyk requested review of this revision.Dec 4 2021, 1:37 PM
amyk created this revision.

My initial thought with this patch was to add the ability to supply the list of projects by a semicolon delimited
list with the --runtimes option, but of course, feedback is welcome with how exactly we want to achieve this
with test-release.sh and this method may not be the best way.

I primarily wanted to post this Phabricator review to start the discussion on building with
-DLLVM_ENABLE_PROJECTS within the release testing script.

Nitpick: In your commit message, I think you mean LLVM_ENABLE_RUNTIMES, not LLVM_ENABLE_PROJECTS.

I'm fine-ish with this change. Another approach would be D112748, where we automatically route various runtimes to LLVM_ENABLE_RUNTIMES without having to change how test-release.sh is invoked. I'm fine with either, I don't really have a preference.

amyk edited the summary of this revision. (Show Details)Dec 7 2021, 2:03 PM

Nitpick: In your commit message, I think you mean LLVM_ENABLE_RUNTIMES, not LLVM_ENABLE_PROJECTS.

I'm fine-ish with this change. Another approach would be D112748, where we automatically route various runtimes to LLVM_ENABLE_RUNTIMES without having to change how test-release.sh is invoked. I'm fine with either, I don't really have a preference.

Oops - yes, that is right. Thanks for letting me know @ldionne! And sorry for missing the initial Phabricator revision that was put up. I've actually added a few comments to your revision to keep the conversation more contained.

ldionne commandeered this revision.Sep 7 2023, 8:15 AM
ldionne edited reviewers, added: amyk; removed: ldionne.

[Github PR transition cleanup]

Commandeering to close since we did D112748 instead.

Herald added a project: Restricted Project. · View Herald TranscriptSep 7 2023, 8:15 AM
ldionne abandoned this revision.Sep 7 2023, 8:15 AM