[PATCH] D136374: [lld-macho] Don't put entries with less than 2 usages into the common table.
Roger Kim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 1 19:51:32 PDT 2022
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
More information about the llvm-commits
mailing list