Run writeTo() concurrently.
authorRui Ueyama <ruiu@google.com>
Sun, 1 Oct 2017 02:25:34 +0000 (02:25 +0000)
committerRui Ueyama <ruiu@google.com>
Sun, 1 Oct 2017 02:25:34 +0000 (02:25 +0000)
I don't know why we didn't use parallelForEach to call writeTo,
but there should be no reason to not do that, as most writeTo
functions are safe to run concurrently.

llvm-svn: 314616

lld/ELF/Writer.cpp

index 1926f1b..ee9af7c 100644 (file)
@@ -1910,14 +1910,16 @@ template <class ELFT> void Writer<ELFT>::writeSections() {
   // In -r or -emit-relocs mode, write the relocation sections first as in
   // ELf_Rel targets we might find out that we need to modify the relocated
   // section while doing it.
-  for (OutputSection *Sec : OutputSections)
+  parallelForEach(OutputSections, [&](OutputSection *Sec) {
     if (Sec->Type == SHT_REL || Sec->Type == SHT_RELA)
       Sec->writeTo<ELFT>(Buf + Sec->Offset);
+  });
 
-  for (OutputSection *Sec : OutputSections)
+  parallelForEach(OutputSections, [&](OutputSection *Sec) {
     if (Sec != Out::Opd && Sec != EhFrameHdr && Sec->Type != SHT_REL &&
         Sec->Type != SHT_RELA)
       Sec->writeTo<ELFT>(Buf + Sec->Offset);
+  });
 
   // The .eh_frame_hdr depends on .eh_frame section contents, therefore
   // it should be written after .eh_frame is written.