[PATCH] D152162: DWP multithreading

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 15:30:06 PDT 2023


MaskRay added a comment.

+1 for achieving low-hanging fruits and switching to `MCStreamer` before adding parallelism. (@dblaikie seems to have a plan)
After these optimizations, the time proportion of different phases may be different enough that warrants a different design of parallelism.
I think this direction is going to have smaller engineering efforts after both `MCStreamer` migration and parallelism work is done.

I have some reservation that adding parallelism now may increase the long-term engineering efforts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152162



More information about the llvm-commits mailing list