<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Aug 10, 2016 at 12:29 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">grimar added a comment.<br>
<span><br>
In <a href="https://reviews.llvm.org/D23315#510206" rel="noreferrer" target="_blank">https://reviews.llvm.org/D2331<wbr>5#510206</a>, @ruiu wrote:<br>
<br>
> Is there any problem if we group sections just by name?<br>
<br>
<br>
</span>Do you mean group into one output section by name ? That is exactly what this patch do.<br>
For linkerscript's OutputSectionFactory "key" is simple StringRef which is output section name.<br>
And I combine the flags when add more sections.<br></blockquote><div><br></div><div>I meant to say that why don't you do this even in Writer.cpp.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D23315" rel="noreferrer" target="_blank">https://reviews.llvm.org/D2331<wbr>5</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>