<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 11, 2017 at 10:02 AM, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Rui Ueyama via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> ruiu added a comment.<br>
><br>
> We used to merge data for COFF but not anymore because it causes nasty problems that are hard to find and debug. E.g. <a href="https://bugs.chromium.org/p/chromium/issues/detail?id=682773#c24" rel="noreferrer" target="_blank">https://bugs.chromium.org/p/<wbr>chromium/issues/detail?id=<wbr>682773#c24</a><br>
><br>
> Are you sure you don't have that kind of issue?<br>
<br>
</span>It is an explicit command line option, but some users do enable it.<br></blockquote><div><br></div><div>Yeah, I know, but I wonder if you could actually link your program without any hidden run-time errors. We've been using lld with such configuration that all read-only data would be merged, and we didn't notice any error at first. From that experience, I'm worried that this feature can be quite dangerous which looks benign at first but actually creates hidden bugs. If users including you really want this, I wouldn't stop thought.</div></div></div></div>