[PATCH] D144509: [CMake] Bumps minimum version to 3.20.0.
Hans Wennborg via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 17 07:40:59 PDT 2023
hans added a comment.
In D144509#4349399 <https://reviews.llvm.org/D144509#4349399>, @mstorsjo wrote:
> In D144509#4349051 <https://reviews.llvm.org/D144509#4349051>, @hans wrote:
>
>> In D144509#4347562 <https://reviews.llvm.org/D144509#4347562>, @glandium wrote:
>>
>>> FYI, 65429b9af6a2c99d340ab2dcddd41dab201f399c <https://reviews.llvm.org/rG65429b9af6a2c99d340ab2dcddd41dab201f399c> is causing problems on Windows compiler-rt for some reason I haven't identified yet (with cmake 3.25.1). Which suggests for a same version of cmake, this is actually altering its behavior, which I wouldn't have expected...
>>
>> It is indeed unfortunate that this seems to come with such a fundamental behavior change. (we already have https://github.com/llvm/llvm-project/issues/62719) Do you have a repro for the issue you're seeing?
>>
>> Since the breakages are piling up, perhaps it's time to revert this.
>
> Unfortunately, much of this is hard to sort out without actually trying to patch forward on main, since many of these issues don't show up easily in e.g. premerge testing or similar (the patch has been tried many times for months before in that form), so I'd appreciate if we'd give it a little more time to actually try to sort out the issues...
But now we have identified issues and there are reproducers available (e.g. on https://github.com/llvm/llvm-project/issues/62719). Keeping it in main is just disruptive and risks making reverting harder (it's already not a clean revert). I think the policy of reverting back to green still applies.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144509/new/
https://reviews.llvm.org/D144509
More information about the cfe-commits
mailing list