[lld] r264945 - Remove useless unreachable. Switch coverage already gives us this. NFC
Pete Cooper via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 30 15:43:26 PDT 2016
Hmm, i was sure I’ve seen us elide unreachable’s in similar situations before. Happy to revert though if GCC isn’t going to like this.
Cheers,
Pete
> On Mar 30, 2016, at 3:42 PM, David Blaikie <dblaikie at gmail.com> wrote:
>
> This is usually needed to suppress GCC's -Wreturn-type warning.
>
> On Wed, Mar 30, 2016 at 3:34 PM, Pete Cooper via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
> Author: pete
> Date: Wed Mar 30 17:34:37 2016
> New Revision: 264945
>
> URL: http://llvm.org/viewvc/llvm-project?rev=264945&view=rev <http://llvm.org/viewvc/llvm-project?rev=264945&view=rev>
> Log:
> Remove useless unreachable. Switch coverage already gives us this. NFC
>
> Modified:
> lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp
>
> Modified: lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp
> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp?rev=264945&r1=264944&r2=264945&view=diff <http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp?rev=264945&r1=264944&r2=264945&view=diff>
> ==============================================================================
> --- lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp (original)
> +++ lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp Wed Mar 30 17:34:37 2016
> @@ -865,7 +865,6 @@ std::error_code Util::getSymbolTableRegi
> }
> break;
> }
> - llvm_unreachable("atom->scope() unknown enum value");
> }
>
> std::error_code Util::addSymbols(const lld::File &atomFile,
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160330/f8d65692/attachment.html>
More information about the llvm-commits
mailing list