<div class="gmail_quote">On Wed, Jun 30, 2010 at 11:42 AM, Sebastian Redl <span dir="ltr"><<a href="mailto:sebastian.redl@getdesigned.at">sebastian.redl@getdesigned.at</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 06/30/2010 08:36 PM, Eric Christopher wrote:<br>
> Author: echristo<br>
> Date: Wed Jun 30 13:36:32 2010<br>
> New Revision: 107305<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=107305&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=107305&view=rev</a><br>
> Log:<br>
> Add a return to silence a warning.  Alternately a default: return false<br>
> would work.<br>
><br>
> Sebastian: figure out which one you want :)<br>
><br>
</div>GCC is confused; the switch is complete on OverloadResult, so what we<br>
really want is an llvm_unreachable. (Is that used in Clang?)<br></blockquote><div><br></div><div>It is, and is a better than return IMO.</div></div>