[PATCH] D109837: cmake: Deprecate config.guess

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 15 13:26:27 PDT 2021


tstellar marked an inline comment as done.
tstellar added a comment.

In D109837#3002372 <https://reviews.llvm.org/D109837#3002372>, @MaskRay wrote:

> I have thought again: we don't even need `LLVM_USE_CONFIG_GUESS`. The warning can just ask the user to run `sh llvm/cmake/config.guess` manually and re-run cmake with -DLLVM_HOST_TRIPLE=, with a caution that the config.guess value may not be correct.

I like this approach, but what if we took it  step further and asked them to download config.guess from https://raw.githubusercontent.com/gcc-mirror/gcc/master/config.guess so that we could remove it from the llvm tree entirely?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109837



More information about the llvm-commits mailing list