[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