[PATCH] D136374: [lld-macho] Don't put entries with less than 2 usages into the common table.

[Roger Kim via Phabricator via llvm-commits]

Roger added inline comments.


================
Comment at: lld/MachO/UnwindInfoSection.cpp:513
+  for (const auto &frequency : encodingFrequencies) {
+    // Don't put encodings with only 1 usage into the table.
+    if (frequency.second < 2) continue;
----------------
Could we include in the comment that the motivation for this if-statement is to match what ld64 is doing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136374