[DirectX] Fix two "not all control paths return a value" warnings

bogner · bogner · commit 7f9ba1994a23 · 2023-07-10T15:34:15.000-07:00
These switch statements are fully covered. Add an llvm_unreachable so that compilers that don't recognize that don't warn. Differential Revision: https://reviews.llvm.org/D154882
diff --git a/llvm/lib/Target/DirectX/DXILResource.cpp b/llvm/lib/Target/DirectX/DXILResource.cpp
@@ -105,6 +105,7 @@ StringRef ResourceBase::getComponentTypeName(ComponentType CompType) {
   case ComponentType::PackedU8x32:
     return "p32u8";
   }
+  llvm_unreachable("All ComponentType enums are handled in switch");
 }
 
 void ResourceBase::printComponentType(Kinds Kind, ComponentType CompType,
@@ -172,6 +173,7 @@ StringRef ResourceBase::getKindName(Kinds Kind) {
   case Kinds::FeedbackTexture2DArray:
     return "fbtex2darray";
   }
+  llvm_unreachable("All Kinds enums are handled in switch");
 }
 
 void ResourceBase::printKind(Kinds Kind, unsigned Alignment, raw_ostream &OS,

Original file line number	Diff line number	Diff line change
`@@ -105,6 +105,7 @@ StringRef ResourceBase::getComponentTypeName(ComponentType CompType) {`
`105`	`105`	`case ComponentType::PackedU8x32:`
`106`	`106`	`return "p32u8";`
`107`	`107`	`}`
	`108`	`+ llvm_unreachable("All ComponentType enums are handled in switch");`
`108`	`109`	`}`
`109`	`110`
`110`	`111`	`void ResourceBase::printComponentType(Kinds Kind, ComponentType CompType,`
`@@ -172,6 +173,7 @@ StringRef ResourceBase::getKindName(Kinds Kind) {`
`172`	`173`	`case Kinds::FeedbackTexture2DArray:`
`173`	`174`	`return "fbtex2darray";`
`174`	`175`	`}`
	`176`	`+ llvm_unreachable("All Kinds enums are handled in switch");`
`175`	`177`	`}`
`176`	`178`
`177`	`179`	`void ResourceBase::printKind(Kinds Kind, unsigned Alignment, raw_ostream &OS,`