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

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 14:08:06 PDT 2016


davide added a comment.

In https://reviews.llvm.org/D26130#583948, @ruiu wrote:

> Well, again, you really need to run a long sequence of computations to get meaningful numbers for benchmark. `clang -help` isn't useful for that purpose.


Agree (as already pointed out in the other thread review). I discussed this a bit with Rafael and I see his point of view, i.e. `clang -help` is definitely a sanity check, and the number of page faults being decreased is a good thing, but I'd really love this to be tested on a startup-time sensitive application.
I mentioned browsers, in particular Firefox, because the Mozilla people spent quite a bit of time thinking about the problem, so you should be able to reuse all their infrastructure (tests etc..). Firefox should link out-the-box with lld, and if it doesn't .. well there's more work to to there =)


https://reviews.llvm.org/D26130





More information about the llvm-commits mailing list