[llvm-commits] [lld] r171645 - /lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Jan 9 13:41:03 PST 2013


Dont' you want to move the llvm_unreachable after the switch?

On 5 January 2013 19:45, Michael J. Spencer <bigcheesegs at gmail.com> wrote:
> Author: mspencer
> Date: Sat Jan  5 18:45:22 2013
> New Revision: 171645
>
> URL: http://llvm.org/viewvc/llvm-project?rev=171645&view=rev
> Log:
> [YAML] Fix -Wcovered-switch-default
>
> Modified:
>     lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp
>
> Modified: lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp
> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp?rev=171645&r1=171644&r2=171645&view=diff
> ==============================================================================
> --- lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp (original)
> +++ lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp Sat Jan  5 18:45:22 2013
> @@ -721,7 +721,6 @@
>          // Eventually we will have an external function to call, similar
>          // to mappingAtoms() and mappingArchive() but implememented
>          // with coresponding file format code.
> -      default:
>          llvm_unreachable("section based YAML not supported yet");
>      }
>    }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list