[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