[PATCH] D13958: Mere SHF_STRING

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 14:02:59 PDT 2015


Hi Rafael,

In my experience, people rarely use linker optimizations that are not on by default. As I recall, nether GCC nor Clang pass -O<n> through to the linker. Is it time to change that?

Thanks again,
Hal

----- Original Message -----
> From: "Rafael Ávila de Espíndola via llvm-commits" <llvm-commits at lists.llvm.org>
> To: ruiu at google.com
> Cc: llvm-commits at lists.llvm.org
> Sent: Friday, October 23, 2015 3:52:15 PM
> Subject: Re: [PATCH] D13958: Mere SHF_STRING
> 
> rafael updated this revision to Diff 38260.
> rafael added a comment.
> 
> This uses fewer classes and makes tail merging optional, controlled
> by -O2 or higher (this is the same option that gold uses).
> 
> The flip side is that this depends on a new llvm class (I will upload
> it to another code review).
> 
> 
> http://reviews.llvm.org/D13958
> 
> Files:
>   ELF/InputFiles.cpp
>   ELF/InputSection.cpp
>   ELF/InputSection.h
>   ELF/OutputSections.cpp
>   ELF/OutputSections.h
>   test/elf2/merge-string-align.s
>   test/elf2/merge-string-error.s
>   test/elf2/merge-string-no-null.s
>   test/elf2/merge-string.s
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory


More information about the llvm-commits mailing list