[PATCH] D24388: [ELF] Merge sections with different access attributes

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 15:20:50 PDT 2016


ruiu added a comment.

As you know we had lots of discussions whether we should merge them or separate them. After several  and forth commits, we are currently not merging sections with different attributes. When the last patch was submitted, the justification of doing it was to handle mergeable sections and other sections properly and also because keeping section writable/executable properties is a good thing. This patch revert that behavior. How would you handle the problems addressed by the last commit?


Repository:
  rL LLVM

https://reviews.llvm.org/D24388





More information about the llvm-commits mailing list