This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Explicit bootstrap options override any passthrough ones.
ClosedPublic

Authored by CarlosAlbertoEnciso on Aug 12 2022, 2:30 AM.

Details

Summary

The https://reviews.llvm.org/D53014 added CMAKE_BUILD_TYPE to
the list of BOOTSTRAP_DEFAULT_PASSTHROUGH variables.

The downside is that both stage-1 and stage-2 configurations
are the same. So it is not possible to build different stage
configurations.

This patch allow explicit bootstrap options to override any
passthrough ones.

For instance, the following settings would build:
stage-1 (Release) and stage-2 (Debug)

"-DCMAKE_BUILD_TYPE=Release -DBOOTSTRAP_CMAKE_BUILD_TYPE=Debug"

Diff Detail

Event Timeline

CarlosAlbertoEnciso requested review of this revision.Aug 12 2022, 2:30 AM
CarlosAlbertoEnciso edited the summary of this revision. (Show Details)Aug 15 2022, 1:49 AM
beanz accepted this revision.Aug 15 2022, 8:27 AM

Looks good to me.

This revision is now accepted and ready to land.Aug 15 2022, 8:27 AM

Looks good to me.

Thanks for the review.