[PATCH] D14567: [ELF2] - dont merge .data.rel.ro/.data.rel.ro.local into .data section.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 12 07:48:39 PST 2015


grimar added a comment.

If the main aim to make code shorter then I would suggest next one.

  if (S.startswith(".data.rel.ro")
    return ".data.rel.ro";

The only point to have this sections in output is to split the rw .data to read only part + smaller rw .data for Relro. There are probably no reasons to keep both .data.rel.ro and .data.rel.ro.local for that. Except behavior consistency with gold of-cource.


http://reviews.llvm.org/D14567





More information about the llvm-commits mailing list