[llvm] r236054 - Reuse a lookup in an assert.

Eric Christopher echristo at gmail.com
Tue Apr 28 15:38:35 PDT 2015


Author: echristo
Date: Tue Apr 28 17:38:35 2015
New Revision: 236054

URL: http://llvm.org/viewvc/llvm-project?rev=236054&view=rev
Log:
Reuse a lookup in an assert.

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

Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=236054&r1=236053&r2=236054&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Tue Apr 28 17:38:35 2015
@@ -2269,10 +2269,10 @@ static ArrayRef<MCPhysReg> get64BitArgum
 
   const Function *Fn = MF.getFunction();
   bool NoImplicitFloatOps = Fn->hasFnAttribute(Attribute::NoImplicitFloat);
-  assert(!(MF.getTarget().Options.UseSoftFloat && NoImplicitFloatOps) &&
+  bool isSoftFloat = MF.getTarget().Options.UseSoftFloat;
+  assert(!(isSoftFloat && NoImplicitFloatOps) &&
          "SSE register cannot be used when SSE is disabled!");
-  if (MF.getTarget().Options.UseSoftFloat || NoImplicitFloatOps ||
-      !Subtarget->hasSSE1())
+  if (isSoftFloat || NoImplicitFloatOps || !Subtarget->hasSSE1())
     // Kernel mode asks for SSE to be disabled, so there are no XMM argument
     // registers.
     return None;





More information about the llvm-commits mailing list