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

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 14:15:00 PDT 2022


int3 added a comment.

That said, I'm not super opposed to making this change. But I'm wondering if we could at least reuse the compact-unwind.s test instead of adding a new file -- this seems like a lot of additional testing for a trivial behavior change. Or will your upcoming patch not fit into that test file?


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