[PATCH] D105786: [lld-macho][nfc] Avoid using std::map for PlatformKinds

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 11 14:00:54 PDT 2021


tschuett added a comment.

switch statements are too verbose? In the `removeSimulator` case, I would expect it to be shorter and more readable. And why is `removeSimulator`not somewhere in llvm Binaryformat?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105786



More information about the llvm-commits mailing list