<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 13, 2013 at 7:21 PM, Duncan Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Nov 13, 2013, at 3:33 PM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>


<br>
>   case NCR_Error:<br>
>     llvm::errs() << "SymbolTable: error while merging " << name << "\n";<br>
> -    // FALLTHRU<br>
> -  default:<br>
> -    llvm::report_fatal_error("SymbolTable::addByName(): unhandled switch clause");<br>
>   }<br>
<br>
</div>Was the change in behavior intended, or should you add a new report_fatal_error() call?<br>
</blockquote></div><br></div><div class="gmail_extra">This change is kind of intended. If something wrong in the input, we sometimes terminate the linker immediately, and sometimes continue processing after printing an error with llvm::err(). Previously the linker is terminated with pretty vague error message ("unhandled switch clause"). With this patch it proceed after printing an error. I was OK in either way, so I just removed default label. I should have made this point clear in the commit log. Sorry about that.</div>

</div>