[cfe-commits] r107305 - /cfe/trunk/lib/Sema/SemaOverload.cpp

[Chandler Carruth]

On Wed, Jun 30, 2010 at 11:42 AM, Sebastian Redl <
sebastian.redl at getdesigned.at> wrote:

> On 06/30/2010 08:36 PM, Eric Christopher wrote:
> > Author: echristo
> > Date: Wed Jun 30 13:36:32 2010
> > New Revision: 107305
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=107305&view=rev
> > Log:
> > Add a return to silence a warning.  Alternately a default: return false
> > would work.
> >
> > Sebastian: figure out which one you want :)
> >
> GCC is confused; the switch is complete on OverloadResult, so what we
> really want is an llvm_unreachable. (Is that used in Clang?)
>

It is, and is a better than return IMO.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20100630/b1e904f0/attachment.html>