From cd1dc7f15d637b42067546e658574237cd0f0d46 Mon Sep 17 00:00:00 2001 From: Mikael Holmen Date: Thu, 5 Mar 2020 09:29:55 +0100 Subject: [PATCH] [AST] Replace assert with llvm_unreachable to silence compiler warning New code added in ec3060c72de6 looked like + case TemplateName::NameKind::OverloadedTemplate: + assert(false && "overloaded templates shouldn't survive to here."); + default: If compiling without asserts we then got a warning about unannotated fallthrough from the case into the default. Change the assert into an llvm_unreachable to silence the warning. --- clang/lib/AST/TemplateName.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clang/lib/AST/TemplateName.cpp b/clang/lib/AST/TemplateName.cpp index afabc57..3b8ae06 100644 --- a/clang/lib/AST/TemplateName.cpp +++ b/clang/lib/AST/TemplateName.cpp @@ -185,7 +185,7 @@ TemplateNameDependence TemplateName::getDependence() const { D |= TemplateNameDependence::UnexpandedPack; break; case TemplateName::NameKind::OverloadedTemplate: - assert(false && "overloaded templates shouldn't survive to here."); + llvm_unreachable("overloaded templates shouldn't survive to here."); default: break; } -- 2.7.4