[Lldb-commits] [PATCH] Request for review: Fix build-llvm.pl

Greg Clayton gclayton at apple.com
Mon Dec 2 09:40:04 PST 2013


Looks good.

On Dec 1, 2013, at 10:58 AM, Jean-Daniel Dupas <devlists at shadowlab.org> wrote:

> Jean-Daniel added you to the CC list for the revision "Request for review: Fix build-llvm.pl".
> 
> - Stop to try to rebuild llvm on each invocation by removing the invalid library entry libLLVMArchive.a which no longer exists.
> - Remove the useless ranlib invocation. "libtools -static" automatically takes care of the archive table of content.
> 
> http://llvm-reviews.chandlerc.com/D2296
> 
> Files:
>  scripts/build-llvm.pl
> 
> Index: scripts/build-llvm.pl
> ===================================================================
> --- scripts/build-llvm.pl
> +++ scripts/build-llvm.pl
> @@ -64,7 +64,6 @@
>     "$llvm_configuration/lib/libclangSema.a",
>     "$llvm_configuration/lib/libclangSerialization.a",
>     "$llvm_configuration/lib/libLLVMAnalysis.a",
> -    "$llvm_configuration/lib/libLLVMArchive.a",
>     "$llvm_configuration/lib/libLLVMARMAsmParser.a",
>     "$llvm_configuration/lib/libLLVMARMAsmPrinter.a",
>     "$llvm_configuration/lib/libLLVMARMCodeGen.a",
> @@ -205,7 +204,7 @@
>                 if (!-d $llvm_dstroot_arch_bin)
>                 {
>                     do_command ("mkdir -p '$llvm_dstroot_arch_bin'", "making llvm build arch bin directory '$llvm_dstroot_arch_bin'", 1);
> -                    my @tools = ("ar", "nm", "ranlib", "strip", "lipo", "ld", "as");
> +                    my @tools = ("ar", "nm", "strip", "lipo", "ld", "as");
>                     my $script_mode = 0755;
>                     my $prog;
>                     for $prog (@tools)
> @@ -390,7 +389,6 @@
>     }
>     close (FILES);
>     do_command ("libtool -static -o '$arch_output_file' -filelist '$files'");
> -    do_command ("ranlib '$arch_output_file'");
> 
>     foreach $object_dir (@object_dirs)
>     {
> <D2296.1.patch>_______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits




More information about the lldb-commits mailing list