[clang-tools-extra] fceec8d - Fix MSVC "not all control paths return a value" warning. NFC.

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 5 02:40:11 PST 2022


Author: Simon Pilgrim
Date: 2022-12-05T10:39:54Z
New Revision: fceec8dcd8da66fa611e7fd104052fa37b65aecb

URL: https://github.com/llvm/llvm-project/commit/fceec8dcd8da66fa611e7fd104052fa37b65aecb
DIFF: https://github.com/llvm/llvm-project/commit/fceec8dcd8da66fa611e7fd104052fa37b65aecb.diff

LOG: Fix MSVC "not all control paths return a value" warning. NFC.

Added: 
    

Modified: 
    clang-tools-extra/include-cleaner/lib/LocateSymbol.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/include-cleaner/lib/LocateSymbol.cpp b/clang-tools-extra/include-cleaner/lib/LocateSymbol.cpp
index cdfe5fda5e95f..c8b6cc77eb148 100644
--- a/clang-tools-extra/include-cleaner/lib/LocateSymbol.cpp
+++ b/clang-tools-extra/include-cleaner/lib/LocateSymbol.cpp
@@ -53,6 +53,7 @@ std::vector<SymbolLocation> locateSymbol(const Symbol &S) {
   case Symbol::Macro:
     return {SymbolLocation(S.macro().Definition)};
   }
+  llvm_unreachable("Unknown Symbol::Kind enum");
 }
 
 } // namespace clang::include_cleaner


        


More information about the cfe-commits mailing list