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

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 06:29:43 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:
> grimar wrote:
> > 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.
> I have an idea how to put all code unrelated to paired relocation to something like `adjustMipsSymVA` and allow `getLocalPaired` to handle only "paired" relocations. I can provide a patch tomorrow.
> 
Ok.


http://reviews.llvm.org/D18083





More information about the llvm-commits mailing list