[PATCH] D112748: [release] Use the Bootstrapping build for building LLVM releases

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 8 14:30:38 PST 2021


ldionne added a comment.

In D112748#3097513 <https://reviews.llvm.org/D112748#3097513>, @phosek wrote:

> In D112748#3094557 <https://reviews.llvm.org/D112748#3094557>, @ldionne wrote:
>
>> @phosek With this patch as-is, we end up not building the runtimes at all because `install-runtimes` isn't executed when we run `ninja -C build`. I *think* that's because `install-runtimes` is not added to the `ALL` target in CMake. Is that voluntary or is it an oversight?
>
> That's because nobody ever asked for it. From what I know, all users of the bootstrapping build use the distribution support <https://llvm.org/docs/BuildingADistribution.html>.
>
> I'm fine including `install-runtimes` in the `ALL` target, but I also think that releases should ideally be using distribution support.

Oh, well, in that case, I think we should definitely be using the distribution support. @phosek any appetite for commandeering this to migrate `test-release.sh` to a proper distribution build?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112748/new/

https://reviews.llvm.org/D112748



More information about the llvm-commits mailing list