<div dir="ltr">+Petr just in case he has something to say (he added the code for max-page-size.)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 30, 2016 at 12:18 PM, Adhemerval Zanella <span dir="ltr"><<a href="mailto:adhemerval.zanella@linaro.org" target="_blank">adhemerval.zanella@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zatrazz added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D25079#557611" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25079#557611</a>, @peter.smith wrote:<br>
<br>
> - LLD doesn't have a concept of common-page-size and max-page-size so if 64k page size is chosen all output sections are aligned to 64k boundary as well as program segments.<br>
<br>
<br>
</span>AFAIK LLD does have a 'max-page-size' option and it exactly overrides the target's default value (check ELF/Driver.cpp:665). That's why I think this change in general is not wrong, but as you said it will add some possible issues to system with uses 4K as default.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D25079" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25079</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>