r320297 - Fix MSVC 'not all control paths return a value' warning

Alex L via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 11 09:27:30 PST 2017


Thanks!

On 10 December 2017 at 03:05, Simon Pilgrim via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: rksimon
> Date: Sun Dec 10 03:05:14 2017
> New Revision: 320297
>
> URL: http://llvm.org/viewvc/llvm-project?rev=320297&view=rev
> Log:
> Fix MSVC 'not all control paths return a value' warning
>
> Modified:
>     cfe/trunk/lib/Driver/ToolChains/Darwin.cpp
>
> Modified: cfe/trunk/lib/Driver/ToolChains/Darwin.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/
> ToolChains/Darwin.cpp?rev=320297&r1=320296&r2=320297&view=diff
> ============================================================
> ==================
> --- cfe/trunk/lib/Driver/ToolChains/Darwin.cpp (original)
> +++ cfe/trunk/lib/Driver/ToolChains/Darwin.cpp Sun Dec 10 03:05:14 2017
> @@ -1230,6 +1230,7 @@ struct DarwinPlatform {
>      case DeploymentTargetEnv:
>        return (llvm::Twine(EnvVarName) + "=" + OSVersion).str();
>      }
> +    llvm_unreachable("Unsupported Darwin Source Kind");
>    }
>
>    static DarwinPlatform createOSVersionArg(DarwinPlatformKind Platform,
>
>
> _______________________________________________
> 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/20171211/4a0d467b/attachment.html>


More information about the cfe-commits mailing list