[PATCH] D44967: Initialize OffsetMap in a known location

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: ELF/SyntheticSections.cpp:2578-2579
     MS->finalizeContents();
+    parallelForEach(MS->Sections,
+                    [](MergeInputSection *Sec) { Sec->initOffsetMap(); });
+  }
----------------
Is the point of this patch to parallelize `initOffsetMap`?

How many sections do you usually have MS->Sections? I thought it's not many.

Did you try parallelizing initOffsetMap itself? We have a huge number of Pieces, so there might be a large room for parallelization.


https://reviews.llvm.org/D44967