HA: [PATCH] D15480: [ELF][MIPS] Handle R_MIPS_HI16/LO16 relocations against _gp_disp symbol

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 24 03:22:40 PST 2015


>>I thinking about that place last time.
>>May be we should add A argument to Target->relocateOne()  ?
>>That can help to move that calculation to mips target, since it looks like absence of A inside relocateOne() is the only stoppable for doing that.
>>
>Currently S+A is a single parameter SA. Are you suggesting split that as two parameters, S and A?
>
>I don't know if that makes code simpler, but it may worth trying.

Not sure about splitting. That seems to be more complicated. Just adding A can save from complicating logic outside allowing relocateOne to use it if it is needed for concrete relocation`s code like here for mips.




More information about the llvm-commits mailing list