[PATCH] D79211: [lld-macho] Support pc-relative section relocations

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 3 16:59:13 PDT 2020


MaskRay added inline comments.


================
Comment at: lld/MachO/InputFiles.cpp:174
+        if (sectIdx >= sections.size())
+          fatal("Relocation section index is greater than number of sections");
+        r.target = sections[sectIdx];
----------------
Error messages are usually not capitalized.

It'd be nice to have a test for zero r_symbolnum


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79211





More information about the llvm-commits mailing list