<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 16, 2016 at 7:40 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><span class="">
<p>> <span style="color:rgb(33,33,33);font-size:12pt">This is not a fatal error which immediately let the linker terminate at least. So from the user's point of view, our error sometimes looks like a warning that just prints out a lot of > error messages
 (although it will eventually make the entire link fail at some point in future.)</span></p>
<p><span style="color:rgb(33,33,33);font-size:12pt"><br>
</span></p>
</span><p><span style="color:rgb(33,33,33);font-size:12pt">That is an issue I believe. We probably want to fatal() at some point, say 50 errors for example.</span></p>
<p><span style="color:rgb(33,33,33);font-size:12pt">(That <span style="color:rgb(33,33,33);font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px;background-color:rgb(255,255,255)">
contradicts</span>​ using lld as a lib I think)</span></p></div></blockquote><div>No, we don't want to turn error into fatal in this case. What I was thinking is to dedup error messages in error() so that each error message is displayed at most once. However, this is a different feature and we don't need it now. So this is okay as long as other review points are addressed.<br></div></div></div></div>