[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