[PATCH] D22370: Respect LLVM_HOST_TRIPLE when manually specified

[Luke Drummond via llvm-commits]

ldrumm added a comment.

Sorry Chris, I didn't see this one. It appeared while I was posting my last message

In https://reviews.llvm.org/D22370#486861, @beanz wrote:

> From config-ix.cmake:
>
>   get_host_triple(LLVM_INFERRED_HOST_TRIPLE)
>  
>   set(LLVM_HOST_TRIPLE "${LLVM_INFERRED_HOST_TRIPLE}" CACHE STRING
>       "Host on which LLVM binaries will run")
>
>
> If `LLVM_HOST_TRIPLE` is set manually on the command line the value read from get_host_triple will not overwrite it because the `FORCE` option isn't specified on the set.
>
> Meaning, this change isn't actually needed to do what you're trying to do anyways. What is needed, is fixing the bugs in get_host_triple, so that it doesn't call config.guess on a Windows system.


So that's the root cause. Okay. That makes sense. I'm not running with a POSIX shell, so it's failing to run that script which is `/bin/sh/`. I'll take a look at that.

Thanks for helping me track down the root cause of this.


Repository:
  rL LLVM

https://reviews.llvm.org/D22370