[clang-tools-extra] r303344 - Fix 'not all control paths return a value' warning on windows buildbots.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Thu May 18 05:34:39 PDT 2017


Thanks!

On Thu, May 18, 2017 at 12:48 PM, Simon Pilgrim via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: rksimon
> Date: Thu May 18 05:48:23 2017
> New Revision: 303344
>
> URL: http://llvm.org/viewvc/llvm-project?rev=303344&view=rev
> Log:
> Fix 'not all control paths return a value' warning on windows buildbots.
>
> Modified:
>     clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp
>
> Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.
> cpp
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/
> trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp?rev=303344&r1=303343&r2=
> 303344&view=diff
> ============================================================
> ==================
> --- clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp
> (original)
> +++ clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp
> Thu May 18 05:48:23 2017
> @@ -170,6 +170,7 @@ public:
>          Result = Globs.contains(S) ? Yes : No;
>          return Result == Yes;
>      }
> +    llvm_unreachable("invalid enum");
>    }
>
>  private:
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170518/c80efbc7/attachment.html>


More information about the cfe-commits mailing list