[PATCH] Forward -u to the linker on gnutools toolchains

[Sylvestre Ledru]

Oh. That is an excellent news!
We were also going to implement this.

This missing option represented 35% of the issues in the Debian rebuild.

On 26/06/2014 03:17, Reid Kleckner wrote:
> Closed by commit rL211756 (authored by @rnk).
>
> REPOSITORY
>   rL LLVM
>
> http://reviews.llvm.org/D4304
>
> Files:
>   cfe/trunk/lib/Driver/Tools.cpp
>   cfe/trunk/test/Driver/linux-ld.c
>
> Index: cfe/trunk/lib/Driver/Tools.cpp
> ===================================================================
> --- cfe/trunk/lib/Driver/Tools.cpp
> +++ cfe/trunk/lib/Driver/Tools.cpp
> @@ -7158,6 +7158,7 @@
>    }
>  
>    Args.AddAllArgs(CmdArgs, options::OPT_L);
> +  Args.AddAllArgs(CmdArgs, options::OPT_u);
>  
>    const ToolChain::path_list Paths = ToolChain.getFilePaths();
>  
> Index: cfe/trunk/test/Driver/linux-ld.c
> ===================================================================
> --- cfe/trunk/test/Driver/linux-ld.c
> +++ cfe/trunk/test/Driver/linux-ld.c
> @@ -1119,3 +1119,9 @@
>  // RUN:        --sysroot=%S/Inputs/basic_linux_tree 2>& 1 \
>  // RUN:   | FileCheck --check-prefix=CHECK-PG %s
>  // CHECK-PG: gcrt1.o
> +
> +// GCC forwards -u to the linker.
> +// RUN: %clang -u asdf --target=x86_64-unknown-linux -### %s \
> +// RUN:        --sysroot=%S/Inputs/basic_linux_tree 2>& 1 \
> +// RUN:   | FileCheck --check-prefix=CHECK-u %s
> +// CHECK-u: "-u" "asdf"
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140710/93a594c0/attachment.html>