[PATCH] D13743: Lazily sort ScopesWithImportedEntities to speed up LTO linking.

Ivan Krasin via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 16 13:50:54 PDT 2015


krasin added a comment.

Update: no, Chrome does not use the split dwarf for the official builds. I was wrong:

https://code.google.com/p/chromium/codesearch#chromium/src/build/common.gypi&q=gsplit-dwarf&sq=package:chromium&type=cs&l=3804

At this time, I have no good explanation, why does not findImportedEntitiesForScope get called. I don't think it's blocking the review, since the CL does not change the logic, it only makes the sorting operation lazy.

I will continue to read the code to get an answer for this question, but please take a look at the CL. It's tested: the Chrome binary is linked faster and bytewise the same as without this change.


http://reviews.llvm.org/D13743





More information about the llvm-commits mailing list