[PATCH] D86909: [lld-macho] Initial support for common symbols

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 13:55:47 PDT 2020


MaskRay added inline comments.


================
Comment at: lld/MachO/SymbolTable.cpp:85
+    if (auto *common = dyn_cast<CommonSymbol>(s)) {
+      if (size >= common->size) {
+        common->file = file;
----------------
This is strange. Is it `>`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86909



More information about the llvm-commits mailing list