Don't repeat names in comments and clang-format this function.

llvm-svn: 232375
This commit is contained in:
Rafael Espindola 2015-03-16 14:05:49 +00:00
parent b2d0c02451
commit 19141f2711
1 changed files with 10 additions and 7 deletions

View File

@ -1094,16 +1094,19 @@ static unsigned X86ChooseCmpOpcode(EVT VT, const X86Subtarget *Subtarget) {
}
}
/// X86ChooseCmpImmediateOpcode - If we have a comparison with RHS as the RHS
/// of the comparison, return an opcode that works for the compare (e.g.
/// CMP32ri) otherwise return 0.
/// If we have a comparison with RHS as the RHS of the comparison, return an
/// opcode that works for the compare (e.g. CMP32ri) otherwise return 0.
static unsigned X86ChooseCmpImmediateOpcode(EVT VT, const ConstantInt *RHSC) {
switch (VT.getSimpleVT().SimpleTy) {
// Otherwise, we can't fold the immediate into this comparison.
default: return 0;
case MVT::i8: return X86::CMP8ri;
case MVT::i16: return X86::CMP16ri;
case MVT::i32: return X86::CMP32ri;
default:
return 0;
case MVT::i8:
return X86::CMP8ri;
case MVT::i16:
return X86::CMP16ri;
case MVT::i32:
return X86::CMP32ri;
case MVT::i64:
// 64-bit comparisons are only valid if the immediate fits in a 32-bit sext
// field.