<div dir="ltr">Davide, it was actually Rui suggestion (look at review comments). He wanted to remove the code which handled special case, when output section contained symbols, but didn't contain any input section.<div>I personally don't mind bringing that code back again.</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-08-03 21:23 GMT+03:00 Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Aug 3, 2016 at 6:21 AM, Eugene Leviant via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
><br>
> 2016-08-03 4:47 GMT+03:00 Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>>:<br>
>><br>
>> The patch as is crashes on linkerscript-provide-in-section.s. It is ok<br>
>> to turn into an error if something is wrong, but please don't crash.<br>
>><br>
>> Cheers,<br>
>> Rafael<br>
>><br>
><br>
> Rafael, this patch deletes that test case, it's strange that you've got this<br>
> crash.<br>
><br>
<br>
</span>Deleting a testcase is seldom, if ever, a good thing to do. What's the<br>
reason why did you delete it? Rafael noticed it's crashing, so we<br>
should probably end up fixing that bug as well.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</font></span></blockquote></div><br></div>