<div dir="ltr"><div>On Thu, Mar 3, 2016 at 12:02 PM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><span>
<p><span style="font-size:12pt;color:rgb(33,33,33)">>The spec is pretty much vague at this point -- it is not clear what "the conceptual placement" means and what it should be.</span><br>
</p>
</span><div style="color:rgb(33,33,33)">
<div>
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><span>
<div>> </div>
<div>>That being said, I'm not entirely against setting a file offset to the bss section. But the two if's don't look good. It probably needs some explanation, but the explanation is that "bss >section should point to its conceptual placement", which doesn't
 really explain the reason that much. So, how about this?</div>
<div>> </div>
</span><div><span>
<div>>  for (OutputSectionBase<ELFT> *Sec : OutputSections) {</div>
<div>>    FileOff = alignTo(FileOff, Sec->getAlign());</div>
<div>>    Sec->setFileOffset(FileOff);</div>
<div>>    FileOff += Sec->getSize();</div>
<div>>  }<br>
</div>
<div><br>
</div>
</span><div>That the same how it was before the patch :)<br>
</div>
<div>​But the point was that .bss should not occupy the space in file. Without that for example tescase for the patch would generate 5 magabytes output what we can and want to avoid, don't we ?</div></div></div></div></div></div></div></div></blockquote><div><br></div><div>LGTM</div><div><br></div><div>Huh, I was thinking that getSize() returns zero for the bss section, but that was not true. I'm not still very satisfied with the code, but looks like it is actually a reasonable approach. Thank you for doing this.</div></div></div></div>