[PATCH] D16135: Macro Debug Info support in Clang

[David Blaikie via cfe-commits]

r294676

On Thu, Feb 9, 2017 at 4:05 PM David L. Jones via Phabricator <
reviews at reviews.llvm.org> wrote:

> dlj added inline comments.
>
>
> ================
> Comment at: cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp:125
> +  switch (Status) {
> +  default:
> +    llvm_unreachable("Do not expect to enter a file from current scope");
> ----------------
> As a heads up... this fails under -Werror:
>
> llvm/tools/clang/lib/CodeGen/MacroPPCallbacks.cpp:125:3: error: default
> label in switch which covers all enumeration values
> [-Werror,-Wcovered-switch-default]
>   default:
>   ^
> 1 error generated.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D16135
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170210/48f97001/attachment-0001.html>