<div dir="ltr">I have a mixed feeling about this. Setting familiar values is probably psychologically better because you wouldn't even think about why the values are the same as before. On the other hand side, this is undeniably a cargo cult.<div><br></div><div>That being said, I'm inclined to Rafael's suggestion. We are serious about the linker, and it's not only me who envision that LLD be one of the standard tools in the Unix world. We would be able to simply stop that cargo cult ourselves.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 13, 2015 at 8:26 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> I had originally thought that they had been causing problems, but going back and restoring the original value (but keeping the page-size fix, which definitely is necessary), sill produces a working hello-world binary. Thus, I don't have a good answer for you, except to say that these are the same magic values that ld.gold uses. That having been said, I'd prefer to keep them this way. Poking around in the debugger, this makes the addresses look more like what I'm used to seeing, and FWIW, matching the behavior of existing linkers here seems like a prudent approach.<br>
<br>
</span>So, this is a small example of "cargo coding" I would like to avoid.<br>
<br>
Changing the values only when we actually know why their are needed<br>
seems a much better solution. In the X86_64 case that is how we found<br>
out what the restrictions actually were and have them documented in a<br>
comment.<br>
<br>
I can see why it has to be 64k aligned on ppc, but the original values was too.<br>
<br>
Would you mind reverting the getVAStart part until we actually know of<br>
a reason why a given architecture has to be different?<br>
<br>
Thanks,<br>
Rafael<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>