[llvm] r363543 - Fix clang -Wcovered-switch-default after stack-id change by D60137

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 17 03:20:20 PDT 2019


Author: maskray
Date: Mon Jun 17 03:20:20 2019
New Revision: 363543

URL: http://llvm.org/viewvc/llvm-project?rev=363543&view=rev
Log:
Fix clang -Wcovered-switch-default after stack-id change by D60137

Modified:
    llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp

Modified: llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp?rev=363543&r1=363542&r2=363543&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp (original)
+++ llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp Mon Jun 17 03:20:20 2019
@@ -545,14 +545,13 @@ static unsigned findScratchNonCalleeSave
 }
 
 bool SIFrameLowering::isSupportedStackID(TargetStackID::Value ID) const {
-    switch (ID) {
-    default:
-      return false;
-    case TargetStackID::Default:
-    case TargetStackID::NoAlloc:
-    case TargetStackID::SGPRSpill:
-      return true;
-    }
+  switch (ID) {
+  case TargetStackID::Default:
+  case TargetStackID::NoAlloc:
+  case TargetStackID::SGPRSpill:
+    return true;
+  }
+  llvm_unreachable("Invalid TargetStackID::Value");
 }
 
 void SIFrameLowering::emitPrologue(MachineFunction &MF,




More information about the llvm-commits mailing list