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

[NAKAMURA Takumi via llvm-commits]

llvm-config is a tool for host's libraries, regardless of the target.

With cross compilation of clang/llvm, Both llvm-config (host's, aka
configure's --build) and llvm-config.exe should be built.
See also, http://bb.pgr.jp/builders/i686-mingw32-RA-on-linux/builds/7293
It is using i686-mingw32-g++ on linux.

On Wed, Feb 10, 2016 at 1:54 PM ChrisBieneman <beanz at apple.com> wrote:

>
> > 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
> >
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160210/9f220c97/attachment.html>