jsji created this revision. jsji added a reviewer: svenvh. Herald added subscribers: cfe-commits, Anastasia, yaxunl. Herald added a project: clang. jsji edited the summary of this revision.
-Werror clang build is broken now. tools/clang/lib/Sema/OpenCLBuiltins.inc:11824:5: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default] default: We don't need default now, since all enumeration values are covered. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72707 Files: clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp Index: clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp =================================================================== --- clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp +++ clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp @@ -751,9 +751,7 @@ } // End of switch statement. - OS << " default:\n" - << " llvm_unreachable(\"OpenCL builtin type not handled yet\");\n" - << " } // end of switch (Ty.ID)\n\n"; + OS << " } // end of switch (Ty.ID)\n\n"; // Step 2. // Add ExtVector types if this was a generic type, as the switch statement
Index: clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp =================================================================== --- clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp +++ clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp @@ -751,9 +751,7 @@ } // End of switch statement. - OS << " default:\n" - << " llvm_unreachable(\"OpenCL builtin type not handled yet\");\n" - << " } // end of switch (Ty.ID)\n\n"; + OS << " } // end of switch (Ty.ID)\n\n"; // Step 2. // Add ExtVector types if this was a generic type, as the switch statement
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
