
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none"><!-- p { margin-top: 0px; margin-bottom: 0px; }--></style>

</head>

<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">

<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>

<div style="color: rgb(33, 33, 33);">

<div>

<div dir="ltr">

<div class="gmail_extra">

<div class="gmail_quote">

<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>

<div>

<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>

<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 ?<br>

</div>

<div><br>

</div>

<div>George.<br>

</div>

</div>

</div>

</div>

</div>

</div>

</div>

</body>

</html>