<div dir="ltr"><div><div><div>Just want to add some more clarification here. It looks like for the current lld code it is an expected behaviour to create two sections. The main function to create the section key is ELF/Writer.cpp:createKey():<br> return SectionKey<ELFT::Is64Bits>{OutsecName, Type, Flags, Alignment};<br></div>So there are 4 arguments to generate the key. And even more, there is a small comment about it:<br>  // For SHF_MERGE we create different output sections for each alignment.<br>  // This makes each output section simple and keeps a single level mapping from<br>  // input to output.<br></div>I can't say if it is good in general, but looks like an expected behaviour.<br><br></div>Also two sections can be created even without a linker script. I tried to describe this case in <a href="https://llvm.org/bugs/show_bug.cgi?id=27805" class="" target="_blank" rel="noreferrer">https://llvm.org/bugs/show_bug.cgi?id=27805</a>. In this case there are two .rodata sections with different alignment values. The good thing here is that without linker script another procedure is used to set the VMA/LMA fields (it just go through all the sections). This approach can't be used as is in case of the linker script.<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 19, 2016 at 9:50 AM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu accepted this revision.<br>
ruiu added a reviewer: ruiu.<br>
ruiu added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM<br>
<br>
I'm singing off since this patch itself doesn't at least make things worse, and and the new test make it explicit what we are doing.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D20378" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20378</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>