[PATCH] D137368: [lld-macho] Overhaul map file code

[Nico Weber via Phabricator via llvm-commits]

thakis added a comment.

Sorry, I missed this!

I didn't look in detail, but from a distance, this looks like a great change.

Does this allow us to undo D115416 <https://reviews.llvm.org/D115416>? I dislike that change a bit – parallelizing things by turning for loops into parallelFor() seems much easier to reason about than manually dealing with futures and the like. (I found _that_ change when I wondered why some for loop somewhere wasn't a parallelFor a while ago – I've since forgotten the details).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137368