[PATCH] D19528: [ELF] - Implemented -z combrelocs/nocombreloc.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 05:30:34 PDT 2016


grimar added a comment.

In http://reviews.llvm.org/D19528#425698, @grimar wrote:

> - Sort only by symbols, using stable_sort.
> - Do not create any DT_entries (I am sure that that reduced startup cycles amount by 10%, so going to introduce another patch after landing this and after retesting the perfomance results I got before).


I retested case when we are creating DT_RELACOUNT vs case when we are not (what is currently commited). Tried many times,
but it seems that difference is just a calcualtion error.
I am not sure what caused 10% difference I observed before, currently I have no reproduce for that anymore.

I tried 
sudo schedtool -F -p 99 -a 0x1 -e perf stat -r 1000 ./clang
as well as just LD_DEBUG=statistics ./clang many times

Numbers are about the same in average.

Changes I tested today were:
F1918028: patch.patch <http://reviews.llvm.org/F1918028>


Repository:
  rL LLVM

http://reviews.llvm.org/D19528





More information about the llvm-commits mailing list