[PATCH] D60353: ELF: Add basic partition data structures and behaviours.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 00:15:12 PDT 2019


MaskRay added inline comments.


================
Comment at: lld/ELF/MarkLive.cpp:352
   // Follow the graph to mark all live sections.
-  MarkLive<ELFT>().run();
+  for (unsigned CurPart = 1; CurPart <= Partitions.size(); ++CurPart)
+    MarkLive<ELFT>(CurPart).run();
----------------
MaskRay wrote:
> I'm concerned with the performance.
> 
> Shall we just do one `MarkLive<ELFT>().run();` with all roots and one `MarkLive<ELFT>(1).moveToMain();`?
> 
> Before, we have npart+1 runs of DFS, now we have just 2 DFS.
Or 1, if you can merge `moveToMain()` into `run()`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60353/new/

https://reviews.llvm.org/D60353





More information about the llvm-commits mailing list