<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 23, 2015 at 4:20 AM, Joerg Sonnenberger via llvm-commits <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="">On Sun, Nov 22, 2015 at 07:29:20PM -0800, Rui Ueyama wrote:<br>
> On Sun, Nov 22, 2015 at 8:56 AM, Joerg Sonnenberger via llvm-commits <<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
> > On Sat, Nov 21, 2015 at 10:19:32PM -0000, Rui Ueyama via llvm-commits<br>
> > wrote:<br>
> > > At least, NetBSD loader does not understand PT_GNU_STACK segments and<br>
> > > reject any executables that have the section. This patch makes lld<br>
> > > compatible with such operating systems.<br>
> ><br>
> > It's not the NetBSD loader, it's third party firmware loaders that's the<br>
> > problem. For the NetBSD loader itself, it's just extra noise.<br>
> > Nevertheless, thanks for the update.<br>
> ><br>
><br>
> Wait, if I knew NetBSD is fine with the former output, I might not have<br>
> made this change. I didn't like this change and submitted as a necessary<br>
> evil. What is that firmware? I'm inclined to roll back this patch.<br>
<br>
</span>OpenFirmware. If there is no way to get rid of PT_GNU_STACK, you can't<br>
build bootable kernels. Being forced to get that junk is bad enough as<br>
is.<br></blockquote><div><br></div><div>I don't think so. If most Unix systems recognize the section or at least ignore that, it is worth to be there unconditionally.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
Joerg<br>
_______________________________________________<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></div>