[PATCH] D18083: [ELF] - Refactor of InputSectionBase<ELFT>::relocate()

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 06:22:50 PST 2016


grimar added inline comments.

================
Comment at: ELF/InputSection.cpp:186
@@ +185,3 @@
+    // the relocation's result for local symbol: S + A + GP0 - G.
+    SymVA += File->getMipsGp0();
+    return nullptr;
----------------
atanasyan wrote:
> It is rather unexpected that function named `getLocalPaired` under some conditions do not find the pair but still change SymVA for relocations that never have pairs.
That is something I don't like in it's name. But it returns something that is used as pair.
I dont have better suggestion for it's name currently.


http://reviews.llvm.org/D18083





More information about the llvm-commits mailing list