[llvm] r307484 - [AArch64] Fix -Wimplicit-fallthrough warnings. NFCI.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 8 12:28:24 PDT 2017


Author: rksimon
Date: Sat Jul  8 12:28:24 2017
New Revision: 307484

URL: http://llvm.org/viewvc/llvm-project?rev=307484&view=rev
Log:
[AArch64] Fix -Wimplicit-fallthrough warnings. NFCI.

Add breaks - doesn't affect results as both GPR/FPU both check for 32/64 bit sizes. So will still default to GenericOps in the same way.

Modified:
    llvm/trunk/lib/Target/AArch64/AArch64InstructionSelector.cpp

Modified: llvm/trunk/lib/Target/AArch64/AArch64InstructionSelector.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64InstructionSelector.cpp?rev=307484&r1=307483&r2=307484&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64InstructionSelector.cpp (original)
+++ llvm/trunk/lib/Target/AArch64/AArch64InstructionSelector.cpp Sat Jul  8 12:28:24 2017
@@ -214,6 +214,7 @@ static unsigned selectBinaryOp(unsigned
         return GenericOpc;
       }
     }
+    break;
   case AArch64::FPRRegBankID:
     switch (OpSize) {
     case 32:
@@ -245,7 +246,8 @@ static unsigned selectBinaryOp(unsigned
         return GenericOpc;
       }
     }
-  };
+    break;
+  }
   return GenericOpc;
 }
 
@@ -269,6 +271,7 @@ static unsigned selectLoadStoreUIOp(unsi
     case 64:
       return isStore ? AArch64::STRXui : AArch64::LDRXui;
     }
+    break;
   case AArch64::FPRRegBankID:
     switch (OpSize) {
     case 8:
@@ -280,7 +283,8 @@ static unsigned selectLoadStoreUIOp(unsi
     case 64:
       return isStore ? AArch64::STRDui : AArch64::LDRDui;
     }
-  };
+    break;
+  }
   return GenericOpc;
 }
 




More information about the llvm-commits mailing list