[clang] Fix build issue after c474f8f (PR #141954)

via cfe-commits cfe-commits at lists.llvm.org
Thu May 29 07:14:24 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Victor Lomuller (Naghasan)

<details>
<summary>Changes</summary>

Remove unneeded default label which triggers a warning with -Wcovered-switch-default

---
Full diff: https://github.com/llvm/llvm-project/pull/141954.diff


1 Files Affected:

- (modified) clang/lib/Sema/SemaSPIRV.cpp (-2) 


``````````diff
diff --git a/clang/lib/Sema/SemaSPIRV.cpp b/clang/lib/Sema/SemaSPIRV.cpp
index bd72194b905f5..c27d3fed2b990 100644
--- a/clang/lib/Sema/SemaSPIRV.cpp
+++ b/clang/lib/Sema/SemaSPIRV.cpp
@@ -129,8 +129,6 @@ static bool checkGenericCastToPtr(Sema &SemaRef, CallExpr *Call) {
     AddrSpace = SemaRef.LangOpts.isSYCL() ? LangAS::sycl_private
                                           : LangAS::opencl_private;
     break;
-  default:
-    llvm_unreachable("Invalid builtin function");
   }
   Qual.setAddressSpace(AddrSpace);
   Call->setType(SemaRef.getASTContext().getPointerType(

``````````

</details>


https://github.com/llvm/llvm-project/pull/141954


More information about the cfe-commits mailing list