[PATCH] D85467: [lld-macho] Support larger dylib symbol ordinals in bindings

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 6 19:34:19 PDT 2020


smeenai added a comment.

I'm not super opposed to not having a test. It doesn't seem too terrible to write one though, just repetitive: we could produce 15 dylibs from the same source file and just have the 16th be different (to confirm the binding), right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85467



More information about the llvm-commits mailing list