commit ff009ad1e1152ac6941ea4a85910f125868bd3de
author Chris Lattner <sabre@nondot.org> Tue Apr 19 05:15:59 2011 +0000
committer Chris Lattner <sabre@nondot.org> Tue Apr 19 05:15:59 2011 +0000
tree a0fcfb82de067033ccff22bdd5f90b6b213dd55e
parent b44101c1401c23fb86fe649309f52e823206147d

tidy up


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@129753 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86FastISel.cpp

diff --git a/lib/Target/X86/X86FastISel.cpp b/lib/Target/X86/X86FastISel.cpp
index 0be9dfa..c563a56 100644
--- a/lib/Target/X86/X86FastISel.cpp
+++ b/lib/Target/X86/X86FastISel.cpp
@@ -1535,11 +1535,12 @@
     }
     
     unsigned ArgReg;
+    
+    // Passing bools around ends up doing a trunc to i1 and passing it.
+    // Codegen this as an argument + "and 1".
     if (ArgVal->getType()->isIntegerTy(1) && isa<TruncInst>(ArgVal) &&
         cast<TruncInst>(ArgVal)->getParent() == I->getParent() &&
         ArgVal->hasOneUse()) {
-      // Passing bools around ends up doing a trunc to i1 and passing it.
-      // Codegen this as an argument + "and 1".
       ArgVal = cast<TruncInst>(ArgVal)->getOperand(0);
       ArgReg = getRegForValue(ArgVal);
       if (ArgReg == 0) return false;
@@ -1551,9 +1552,10 @@
                            ArgVal->hasOneUse(), 1);
     } else {
       ArgReg = getRegForValue(ArgVal);
-      if (ArgReg == 0) return false;
     }
 
+    if (ArgReg == 0) return false;
+
     // FIXME: Only handle *easy* calls for now.
     if (CS.paramHasAttr(AttrInd, Attribute::InReg) ||
         CS.paramHasAttr(AttrInd, Attribute::StructRet) ||