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

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 11 10:14:22 PDT 2020


Author: Simon Pilgrim
Date: 2020-08-11T18:13:03+01:00
New Revision: b9aaf32f46494695d1b20c08730c1111536e17f8

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

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

Added: 
    

Modified: 
    clang/lib/Tooling/Transformer/RewriteRule.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/Tooling/Transformer/RewriteRule.cpp b/clang/lib/Tooling/Transformer/RewriteRule.cpp
index 8781bfc66146..89497f67bbb5 100644
--- a/clang/lib/Tooling/Transformer/RewriteRule.cpp
+++ b/clang/lib/Tooling/Transformer/RewriteRule.cpp
@@ -131,6 +131,7 @@ static std::string formatHeaderPath(StringRef Header, IncludeFormat Format) {
   case transformer::IncludeFormat::Angled:
     return ("<" + Header + ">").str();
   }
+  llvm_unreachable("Unknown transformer::IncludeFormat enum");
 }
 
 ASTEdit transformer::addInclude(RangeSelector Target, StringRef Header,


        


More information about the cfe-commits mailing list