[PATCH] D14382: [ELF2] - Basic implementation of -r/--relocatable

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 5 10:10:09 PST 2015


grimar added a comment.

In http://reviews.llvm.org/D14382#282400, @ruiu wrote:

> I'd like to see a patch to apply (and reduce) relocations. This patch seems generally good, but there's a chance that the fields and functions added in this patch will not make sense with the follow-up patch. We don't have to combine all changes into one patch, but can split into a series of patches.


Well I tried to build some background. All added functions/variables are mostly used for -r option which implementation is a subject to change anyways since it is initial. 
I understand the core idea you want to see in this patch at first though. So if reducing relocations is needed for it initial commit, I can handle that I think. Currently I dont know the valid test case for that yet (which exactly relocations can be applied during relocatable output linking). If you could say what can I use then that info will be really helpfull, if not I`ll try to investigate that tomorrow.


http://reviews.llvm.org/D14382





More information about the llvm-commits mailing list