[PATCH] D35005: [ELF] - Do not apply relocations to .debug_ranges when building -gdb-index

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: ELF/SyntheticSections.cpp:1821-1822
+      return false;
+    if (S == ".rela.debug_ranges" || S == ".rel.debug_ranges")
+      return relocateDebugRanges(RelSec, Map);
+    return false;
----------------
dblaikie wrote:
> Any particular reason this would only be done for ranges sections and not for other sections?
I tried to focus on feature itself and keep patch as short as possible for initial review. 
I think that can be done for all sections required for building .gdb_index (though I did not check by myself yet).
At least I do not know any reason why it should not work atm.


https://reviews.llvm.org/D35005