[llvm] r232375 - Don't repeat names in comments and clang-format this function.

Rafael Espindola rafael.espindola at gmail.com
Mon Mar 16 07:05:49 PDT 2015


Author: rafael
Date: Mon Mar 16 09:05:49 2015
New Revision: 232375

URL: http://llvm.org/viewvc/llvm-project?rev=232375&view=rev
Log:
Don't repeat names in comments and clang-format this function.

Modified:
    llvm/trunk/lib/Target/X86/X86FastISel.cpp

Modified: llvm/trunk/lib/Target/X86/X86FastISel.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86FastISel.cpp?rev=232375&r1=232374&r2=232375&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86FastISel.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86FastISel.cpp Mon Mar 16 09:05:49 2015
@@ -1094,16 +1094,19 @@ static unsigned X86ChooseCmpOpcode(EVT V
   }
 }
 
-/// 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.





More information about the llvm-commits mailing list