[PATCH] D13198: Fix `llvm-config` to adapt to the install environment.

ChrisBieneman via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 20:54:14 PST 2016


> On Feb 9, 2016, at 5:21 PM, NAKAMURA Takumi <geek4civic at gmail.com> wrote:
> 
> chapuni added a subscriber: chapuni.
> 
> ================
> Comment at: llvm/trunk/tools/llvm-config/llvm-config.cpp:331
> @@ +330,3 @@
> +    StaticPrefix, StaticDir = "lib";
> +  const Triple HostTriple(Triple::normalize(LLVM_DEFAULT_TARGET_TRIPLE));
> +  if (HostTriple.isOSWindows()) {
> ----------------
> The behavior should be controlled not by target, but by host. Fixed in r260343.
> Consider if host=linux, target=i686-win32.

I don't follow. This is the logic that controls file extensions. If my host is linux, but I'm cross compiling for Windows, I would expect the dynamic libraries to be "dll" files. Correct?

-Chris

> 
> 
> Repository:
>  rL LLVM
> 
> http://reviews.llvm.org/D13198
> 
> 
> 


More information about the llvm-commits mailing list