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

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 4 01:58:41 PDT 2024


Author: Simon Pilgrim
Date: 2024-10-04T09:54:13+01:00
New Revision: 992e75403f417d46ce9015147ff49d67009e736c

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

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

Added: 
    

Modified: 
    clang-tools-extra/clangd/refactor/tweaks/SwapBinaryOperands.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clangd/refactor/tweaks/SwapBinaryOperands.cpp b/clang-tools-extra/clangd/refactor/tweaks/SwapBinaryOperands.cpp
index 1602093e3d2893..9ad0089a5d0359 100644
--- a/clang-tools-extra/clangd/refactor/tweaks/SwapBinaryOperands.cpp
+++ b/clang-tools-extra/clangd/refactor/tweaks/SwapBinaryOperands.cpp
@@ -126,6 +126,7 @@ BinaryOperatorKind swapOperator(const BinaryOperatorKind Opcode) {
   case BinaryOperatorKind::BO_OrAssign:
     return Opcode;
   }
+  llvm_unreachable("Unknown BinaryOperatorKind enum");
 }
 
 /// Swaps the operands to a binary operator


        


More information about the cfe-commits mailing list