<div dir="ltr">Buildbot sanitizer-x86_64-linux-bootstrap reported an error (build 418):<div><div><br></div><div>/home/dtoolsbot/build/sanitizer-x86_64-linux/build/llvm/tools/clang/lib/Sema/SemaLookup.cpp:1874:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default]</div>
<div>  default:</div><div>  ^</div><div>1 error generated.</div></div><div><br></div><div>You are absolutely right, it is<span style="font-family:arial,sans-serif;font-size:13px"> </span><span style="font-family:arial,sans-serif;font-size:13px">-Wcovered-switch-default.</span></div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/8/29 David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Aug 29, 2013 at 1:20 AM, Serge Pavlov <<a href="mailto:sepavloff@gmail.com">sepavloff@gmail.com</a>> wrote:<br>
> Author: sepavloff<br>
> Date: Thu Aug 29 03:20:07 2013<br>
> New Revision: 189575<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=189575&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=189575&view=rev</a><br>
> Log:<br>
> Removed useless default branch of switch statement.<br>
> The problem was caught by sanitizer build.<br>
<br>
</div>Caught in what way? I'm not sure how any of the sanitizers would help<br>
with this specific issue. Was this just a -Wcovered-switch-default<br>
warning/break?<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Modified:<br>
>     cfe/trunk/lib/Sema/SemaLookup.cpp<br>
><br>
> Modified: cfe/trunk/lib/Sema/SemaLookup.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaLookup.cpp?rev=189575&r1=189574&r2=189575&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaLookup.cpp?rev=189575&r1=189574&r2=189575&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/lib/Sema/SemaLookup.cpp (original)<br>
> +++ cfe/trunk/lib/Sema/SemaLookup.cpp Thu Aug 29 03:20:07 2013<br>
> @@ -1870,9 +1870,6 @@ void Sema::DiagnoseAmbiguousLookup(Looku<br>
>        Diag((*DI)->getLocation(), diag::note_ambiguous_candidate) << *DI;<br>
>      break;<br>
>    }<br>
> -<br>
> -  default:<br>
> -    llvm_unreachable("unknown ambiguity kind");<br>
>    }<br>
>  }<br>
><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Thanks,<br>--Serge<br>
</div>