[PATCH] D26130: [ELF] - Implemented --symbol-ordering-file option.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 02:23:50 PDT 2016


>I just finished a run of firefox.
>
>Unfortunately it is substantially harder to test than clang. What is needed is
>
>* Create a test.html file with window.close() so that we have a
>firefox invocation we can time.

In my experiments I also used this pluging:
https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Measuring_add-on_startup_performance
It seems to work, but no noticable change (for sections list) anyways as I mentioned in previous threads.

>* Subtract the xul load address since perf -D prints raw values.
>* The link with 33k symbols in the order file then takes about 8 minutes.
>
>And after all that there was no significant change in performance.
>Assuming the patch can be cleaned up and optimized I think there is
>enough justification for it.
>
>George, you should probably post your updated icegrind plugin
>somewhere. The two main advantages it should have is not requiring a
>crazy high sampling rate and also ordering data sections.

Updated patch to r16110 of valgrind (latest). It is in attachment.

>Cheers,
>Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icegrind_r16110.patch
Type: application/octet-stream
Size: 489919 bytes
Desc: icegrind_r16110.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161102/5d7d06f4/attachment-0001.obj>


More information about the llvm-commits mailing list