<div dir="ltr"><div><span style="font-size:12.8px">> * Assigning offsets as we assign sections to output sections</span><br></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Please keep in mind that output section size and symbol values depend on output section VA, </span></div><div><span style="font-size:12.8px">because you can have the following kind of expression within section description:</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">. = ALIGN(0x8);</span></div><div><span style="font-size:12.8px"><br></span></div>I will wait for Rui, just in case he wants something to be fixed before landing this patch.<div>Thanks!</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-08-30 18:26 GMT+03:00 Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> I think the code can be simplified further (I am experimenting with it), but this is a step in the right direction.<br>
<br>
</span>Some things I tried:<br>
<br>
* Assigning offsets as we assign sections to output sections. That is<br>
too early as '.' needs the address, and for that we need the alignment<br>
of the output section.<br>
<br>
* Storing in InputSectionDescription which sections it found. With<br>
this assignOffsets can be simpler as we visit the assignments and<br>
sections in the right order. This is probably worth it, but handling<br>
the orphan sections is still a bit annoying.<br>
<br>
I will probably send the second patch for review once this one is in.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>